Fix Q(Quick)Window on eglfs

The call to QPlatformWindow::setRect (for storing into d->rect) was in the wrong place. It has to be the potentially-overridden value. Amends 3a31c70 Task-number: QTBUG-57608 Change-Id: Id3c35e2dc178f7bd2f9643e8ae4754c8f2f39240 Reviewed-by: Friedemann Kleint <[email protected]>
Someguynamedpie · Aug 3, 2017 · 51446c3 · 51446c3
1 parent d12d294
commit 51446c3
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/src/plugins/platforms/eglfs/api/qeglfswindow.cpp b/src/plugins/platforms/eglfs/api/qeglfswindow.cpp
@@ -235,18 +235,15 @@ void QEglFSWindow::setVisible(bool visible)
 
 void QEglFSWindow::setGeometry(const QRect &r)
 {
-    // Persist the requested rect, like a normal setGeometry call
-    QPlatformWindow::setGeometry(r);
-
-    // Take care of WM behavior, constrain/modify geometry
     QRect rect = r;
     if (m_flags.testFlag(HasNativeWindow))
         rect = screen()->availableGeometry();
 
-    // React to the setGeometry, as if from a WM callback
-    QRect lastReportedGeometry = qt_window_private(window())->geometry;
+    QPlatformWindow::setGeometry(rect);
+
     QWindowSystemInterface::handleGeometryChange(window(), rect);
 
+    const QRect lastReportedGeometry = qt_window_private(window())->geometry;
     if (rect != lastReportedGeometry)
         QWindowSystemInterface::handleExposeEvent(window(), QRect(QPoint(0, 0), rect.size()));
 }