From 29901fc2a97eedd3c914f807d1819c9ea7e69973 Mon Sep 17 00:00:00 2001
From: Sine Striker <trueful@163.com>
Date: 周一, 25 12月 2023 17:53:34 +0800
Subject: [PATCH] Optimize Windows 10 border handling

---
 src/widgets/widgetwindowagent_win.cpp |   57 +++++++++++++++++++++++++++++++++++++++++++++------------
 1 files changed, 45 insertions(+), 12 deletions(-)

diff --git a/src/widgets/widgetwindowagent_win.cpp b/src/widgets/widgetwindowagent_win.cpp
index dc35a17..9e49d38 100644
--- a/src/widgets/widgetwindowagent_win.cpp
+++ b/src/widgets/widgetwindowagent_win.cpp
@@ -1,9 +1,10 @@
 #include "widgetwindowagent_p.h"
 
+#include <QtCore/QDebug>
+#include <QtCore/QDateTime>
 #include <QtGui/QPainter>
 
 #include <QWKCore/qwindowkit_windows.h>
-#include <QWKCore/qwkconfig.h>
 #include <QWKCore/private/nativeeventfilter_p.h>
 
 namespace QWK {
@@ -12,19 +13,21 @@
 
     class WidgetBorderHandler;
 
-    class WidgetBorderHandler : public QObject, public NativeEventFilter {
+    class WidgetBorderHandler : public QObject, public NativeEventFilter, public SharedEventFilter {
     public:
         explicit WidgetBorderHandler(QWidget *widget, AbstractWindowContext *ctx)
             : QObject(ctx), widget(widget), ctx(ctx) {
             widget->installEventFilter(this);
 
             ctx->installNativeEventFilter(this);
+            ctx->installSharedEventFilter(this);
+
             updateGeometry();
         }
 
         inline bool isNormalWindow() const {
-            return widget->windowState() &
-                   (Qt::WindowMinimized | Qt::WindowMaximized | Qt::WindowFullScreen);
+            return !(widget->windowState() &
+                     (Qt::WindowMinimized | Qt::WindowMaximized | Qt::WindowFullScreen));
         }
 
         void updateGeometry() {
@@ -66,19 +69,48 @@
                     }
                     break;
                 }
+#  if 0
+                case WM_ACTIVATE: {
+                    if (LOWORD(msg->wParam) == WA_INACTIVE) {
+                        // 绐楀彛澶卞幓婵�娲荤姸鎬�
+                    } else {
+                        // 绐楀彛琚縺娲�
+                    }
+                    break;
+                }
+#  endif
 
-                case WM_PAINT: {
+                default:
+                    break;
+            }
+            return false;
+        }
+
+        bool sharedEventFilter(QObject *obj, QEvent *event) override {
+            Q_UNUSED(obj)
+
+            auto window = widget->windowHandle();
+            if (event->type() == QEvent::Expose) {
+                // Qt will absolutely send a QExposeEvent to the QWindow when it receives a
+                // WM_PAINT message. When the control flow enters the expose handler, Qt must
+                // have already called BeginPaint() and it's the best time for us to draw the
+                // top border.
+                auto ee = static_cast<QExposeEvent *>(event);
+                if (isNormalWindow() && window->isExposed() && !ee->region().isNull()) {
+                    // Friend class helping to call `event`
+                    class HackedWindow : public QWindow {
+                    public:
+                        friend class QWK::WidgetBorderHandler;
+                    };
+
                     // Let Qt paint first
-                    m_dispatcher->resume(eventType, message, result);
+                    static_cast<HackedWindow *>(window)->event(event);
 
                     // Upon receiving the WM_PAINT message, Qt will redraw the entire view, and we
                     // must wait for it to finish redrawing before drawing this top border area
                     ctx->virtual_hook(AbstractWindowContext::DrawWindows10BorderHook2, nullptr);
                     return true;
                 }
-
-                default:
-                    break;
             }
             return false;
         }
@@ -87,6 +119,9 @@
             Q_UNUSED(obj)
             switch (event->type()) {
                 case QEvent::UpdateRequest: {
+                    if (!isNormalWindow())
+                        break;
+
                     // Friend class helping to call `event`
                     class HackedWidget : public QWidget {
                     public:
@@ -123,16 +158,14 @@
         QWidget *widget;
         AbstractWindowContext *ctx;
     };
-#endif
 
     void WidgetWindowAgentPrivate::setupWindows10BorderWorkaround() {
-#if QWINDOWKIT_CONFIG(ENABLE_WINDOWS_SYSTEM_BORDER)
         // Install painting hook
         auto ctx = context.get();
         if (ctx->property("needBorderPainter").toBool()) {
             std::ignore = new WidgetBorderHandler(hostWidget, ctx);
         }
-#endif
     }
+#endif
 
 }

--
Gitblit v1.9.1