From 4e7058084d51193e3c975bcddbe749e7fd29e356 Mon Sep 17 00:00:00 2001
From: Sine Striker <trueful@163.com>
Date: 周日, 19 5月 2024 23:29:49 +0800
Subject: [PATCH] Add check of maximum button hint

---
 src/core/contexts/abstractwindowcontext.cpp |   97 +++++++++++++-----------------------------------
 1 files changed, 27 insertions(+), 70 deletions(-)

diff --git a/src/core/contexts/abstractwindowcontext.cpp b/src/core/contexts/abstractwindowcontext.cpp
index 8fa851a..71818c2 100644
--- a/src/core/contexts/abstractwindowcontext.cpp
+++ b/src/core/contexts/abstractwindowcontext.cpp
@@ -7,65 +7,10 @@
 #include <QtGui/QPen>
 #include <QtGui/QPainter>
 #include <QtGui/QScreen>
-#include <memory>
 
 #include "qwkglobal_p.h"
 
 namespace QWK {
-
-    namespace {
-
-        class WinIdChangeEventFilter : public QObject {
-        public:
-            explicit WinIdChangeEventFilter(QObject *widget, AbstractWindowContext *ctx,
-                                            QObject *parent = nullptr)
-                : QObject(parent), ctx(ctx) {
-                widget->installEventFilter(this);
-            }
-
-        protected:
-            bool eventFilter(QObject *obj, QEvent *event) override {
-                Q_UNUSED(obj)
-                if (event->type() == QEvent::WinIdChange) {
-                    ctx->notifyWinIdChange();
-                }
-                return false;
-            }
-
-        protected:
-            AbstractWindowContext *ctx;
-        };
-
-        class WindowEventFilter : public QObject {
-        public:
-            explicit WindowEventFilter(QWindow *window, AbstractWindowContext *ctx,
-                                       QObject *parent = nullptr)
-                : QObject(parent), ctx(ctx), window(window) {
-                if (window)
-                    window->installEventFilter(this);
-            }
-
-            inline void setWindow(QWindow *win) {
-                if (auto oldWin = window.data(); oldWin) {
-                    oldWin->removeEventFilter(this);
-                }
-                window = win;
-                if (win) {
-                    win->installEventFilter(this);
-                }
-            }
-
-        protected:
-            bool eventFilter(QObject *obj, QEvent *event) override {
-                return ctx->sharedDispatch(obj, event);
-            }
-
-        protected:
-            AbstractWindowContext *ctx;
-            QPointer<QWindow> window;
-        };
-
-    }
 
     AbstractWindowContext::AbstractWindowContext() = default;
 
@@ -77,16 +22,9 @@
         }
         m_host = host;
         m_delegate.reset(delegate);
-        m_winIdChangeEventFilter = std::make_unique<WinIdChangeEventFilter>(host, this);
-        m_windowEventFilter = std::make_unique<WindowEventFilter>(m_windowHandle, this);
-
-        m_windowHandle = m_delegate->hostWindow(m_host);
-        if (m_windowHandle) {
-            winIdChanged();
-        }
+        m_winIdChangeEventFilter.reset(delegate->createWinIdEventFilter(host, this));
+        notifyWinIdChange();
     }
-
-
 
     bool AbstractWindowContext::setHitTestVisible(const QObject *obj, bool visible) {
         Q_ASSERT(obj);
@@ -213,7 +151,7 @@
     void AbstractWindowContext::virtual_hook(int id, void *data) {
         switch (id) {
             case CentralizeHook: {
-                if (!m_windowHandle)
+                if (!m_windowId)
                     return;
 
                 QRect windowGeometry = m_delegate->getGeometry(m_host);
@@ -227,7 +165,7 @@
             }
 
             case RaiseWindowHook: {
-                if (!m_windowHandle)
+                if (!m_windowId)
                     return;
 
                 m_delegate->setWindowVisible(m_host, true);
@@ -259,18 +197,24 @@
     }
 
     void AbstractWindowContext::notifyWinIdChange() {
-        auto windowEventFilter = static_cast<WindowEventFilter *>(m_windowEventFilter.get());
-        windowEventFilter->setWindow(nullptr);
+        auto oldWinId = m_windowId;
+        m_windowId = m_winIdChangeEventFilter->winId();
 
         // In Qt6, after QWidget::close() is called, the related QWindow's all surfaces and the
         // platform window will be removed, and the WinId will be set to 0. After that, when the
         // QWidget is shown again, the whole things will be recreated again.
         // As a result, we must update our WindowContext each time the WinId changes.
+        if (m_windowHandle) {
+            removeEventFilter(m_windowHandle);
+        }
         m_windowHandle = m_delegate->hostWindow(m_host);
-        winIdChanged();
+
+        if (oldWinId != m_windowId) {
+            winIdChanged(m_windowId, oldWinId);
+        }
 
         if (m_windowHandle) {
-            windowEventFilter->setWindow(m_windowHandle);
+            m_windowHandle->installEventFilter(this);
 
             // Refresh window attributes
             auto attributes = m_windowAttributes;
@@ -281,6 +225,12 @@
                 }
                 m_windowAttributes.insert(it.key(), it.value());
             }
+        }
+
+        // Send to shared dispatchers
+        if (oldWinId != m_windowId) {
+            QEvent e(QEvent::WinIdChange);
+            sharedDispatch(m_host, &e);
         }
     }
 
@@ -315,6 +265,13 @@
         return true;
     }
 
+    bool AbstractWindowContext::eventFilter(QObject *obj, QEvent *event) {
+        if (obj == m_windowHandle && sharedDispatch(obj, event)) {
+            return true;
+        }
+        return QObject::eventFilter(obj, event);
+    }
+
     bool AbstractWindowContext::windowAttributeChanged(const QString &key,
                                                        const QVariant &attribute,
                                                        const QVariant &oldAttribute) {

--
Gitblit v1.9.1