Fix crash caused by setting Tray window screen pointer on the QML side

Seems like Qt sometimes doesn't like the QML window's screen property to be set
to a C++ (QVariant) pointer value, so we use the index: Qt.application.screens[]

See Qt docs: https://doc.qt.io/qt-5/qml-qtquick-window-window.html#screen-prop

This fix returns the matching window's index from the QGuiApplication::screens()
list to the QML side, instead of the window pointer.

Steps to reproduce the crash with the previous code:
- Open the Tray menu and close it a few times,
  or scroll randomly up and down in its activity list.

Tested with Qt 5.12.5

Signed-off-by: Michael Schuster <michael@schuster.ms>

diff --git a/src/gui/systray.cpp b/src/gui/systray.cpp
index 97f19765c31abe9e0f2117d6743826eb9c374d31..ebcefc4df302f4c1066b53412c5df589f24232e5 100644 (file)
--- a/src/gui/systray.cpp
+++ b/src/gui/systray.cpp
@@ -158,9 +158,11 @@ QScreen *Systray::currentScreen() const
     return nullptr;
 }
 
-QVariant Systray::currentScreenVar() const
+int Systray::currentScreenIndex() const
 {
-    return QVariant::fromValue(currentScreen());
+    const auto screens = QGuiApplication::screens();
+    const auto screenIndex = screens.indexOf(currentScreen());
+    return screenIndex > 0 ? screenIndex : 0;
 }
 
 Systray::TaskBarPosition Systray::taskbarOrientation() const

diff --git a/src/gui/systray.h b/src/gui/systray.h
index feb0771aafa2dd9ddba0bd7bfcd589795ad72a15..0cb9249f6c3f926bd505c1a2a2666cd817f3d407 100644 (file)
--- a/src/gui/systray.h
+++ b/src/gui/systray.h
@@ -58,7 +58,7 @@ public:
     Q_INVOKABLE bool syncIsPaused();
     Q_INVOKABLE void setOpened();
     Q_INVOKABLE void setClosed();
-    Q_INVOKABLE QVariant currentScreenVar() const;
+    Q_INVOKABLE int currentScreenIndex() const;
     Q_INVOKABLE QPoint calcTrayIconCenter() const;
     Q_INVOKABLE TaskBarPosition taskbarOrientation() const;
     Q_INVOKABLE QRect taskbarGeometry() const;

diff --git a/src/gui/tray/Window.qml b/src/gui/tray/Window.qml
index 9e11828b8a042082c8a58d970f887d7d4541303a..2ec850d4afb539024e2f0f224d9f5b097cd9cc82 100644 (file)
--- a/src/gui/tray/Window.qml
+++ b/src/gui/tray/Window.qml
@@ -63,7 +63,7 @@ Window {
         onShowWindow: {\r
             accountMenu.close();\r
 \r
-            trayWindow.screen = systrayBackend.currentScreenVar();\r
+            trayWindow.screen = Qt.application.screens[systrayBackend.currentScreenIndex()];\r
 \r
             var position = systrayBackend.computeWindowPosition(trayWindow.width, trayWindow.height)\r
             trayWindow.x = position.x\r