From 326b23bf678301a72cb9d67ac64347dd5ce49a37 Mon Sep 17 00:00:00 2001
From: Yuhang Zhao <zhaoyuhang@rankyee.com>
Date: 周一, 18 12月 2023 14:27:11 +0800
Subject: [PATCH] optimize mac again

---
 src/core/contexts/cocoawindowcontext.mm |   23 ++++++++---------------
 1 files changed, 8 insertions(+), 15 deletions(-)

diff --git a/src/core/contexts/cocoawindowcontext.mm b/src/core/contexts/cocoawindowcontext.mm
index edd2c3a..a8673b8 100644
--- a/src/core/contexts/cocoawindowcontext.mm
+++ b/src/core/contexts/cocoawindowcontext.mm
@@ -207,15 +207,6 @@
         return [nsview window];
     }
 
-    static inline void cleanupProxy() {
-        if (g_proxyList()->isEmpty()) {
-            return;
-        }
-        const auto &data = *g_proxyList();
-        qDeleteAll(data);
-        g_proxyList()->clear();
-    }
-
     static inline NSWindowProxy *ensureWindowProxy(const WId windowId) {
         auto it = g_proxyList()->find(windowId);
         if (it == g_proxyList()->end()) {
@@ -223,12 +214,13 @@
             const auto proxy = new NSWindowProxy(nswindow);
             it = g_proxyList()->insert(windowId, proxy);
         }
-        static bool cleanerInstalled = false;
-        if (!cleanerInstalled) {
-            cleanerInstalled = true;
-            qAddPostRoutine(cleanupProxy);
-        }
         return it.value();
+    }
+
+    static inline void releaseWindowProxy(const WId windowId) {
+        if (const auto proxy = g_proxyList()->take(windowId)) {
+            delete proxy;
+        }
     }
 
     class CocoaWindowEventFilter : public QObject {
@@ -369,7 +361,7 @@
     }
 
     CocoaWindowContext::~CocoaWindowContext() {
-        // TODO: deref something?
+        releaseWindowProxy(windowId);
     }
 
     QString CocoaWindowContext::key() const {
@@ -391,6 +383,7 @@
     }
 
     void CocoaWindowContext::winIdChanged(QWindow *oldWindow, bool destroyed) {
+        releaseWindowProxy(windowId);
         windowId = m_windowHandle->winId();
         ensureWindowProxy(windowId)->setSystemTitleBarVisible(false);
         cocoaWindowEventFilter = std::make_unique<CocoaWindowEventFilter>(this, this);

--
Gitblit v1.9.1