QRhiWidget: Tear down resources before window has changed

We were handling QEvent::WindowChangeInternal, but based on what the
code is doing, what the comments say, and what QQuickWidget does,
the right event should be WindowAboutToChangeInternal.

This fixes a crash when reparenting a QRhiWidget into another widget,
where we could call removeCleanupCallback() on an RHI that was already
gone.

The fact that we have a stale RHI pointer at WindowChangeInternal time
is still problematic, and caused by our call to addCleanupCallback not
happening as it should for the first call to ensureRhi(), but this will
be fixed in a follow up.

Pick-to: 6.7
Change-Id: I054120e97a24a1f74af44b2d251470792f03f7f3
Reviewed-by: Laszlo Agocs <laszlo.agocs@qt.io>

1 files changed, 8 insertions, 1 deletions

diff --git a/tests/auto/widgets/widgets/qrhiwidget/tst_qrhiwidget.cpp b/tests/auto/widgets/widgets/qrhiwidget/tst_qrhiwidget.cpp
index 988f736e4e..9f56c77637 100644
--- a/tests/auto/widgets/widgets/qrhiwidget/tst_qrhiwidget.cpp
+++ b/tests/auto/widgets/widgets/qrhiwidget/tst_qrhiwidget.cpp
@@ -635,12 +635,19 @@ void tst_QRhiWidget::reparent()
     QWidget *windowOne = new QWidget;
     windowOne->resize(1280, 720);
 
-    SimpleRhiWidget *rhiWidget = new SimpleRhiWidget(1, windowOne);
+    SimpleRhiWidget *rhiWidget = new SimpleRhiWidget(1);
     rhiWidget->setApi(api);
     rhiWidget->resize(800, 600);
     QSignalSpy frameSpy(rhiWidget, &QRhiWidget::frameSubmitted);
     QSignalSpy errorSpy(rhiWidget, &QRhiWidget::renderFailed);
 
+    rhiWidget->show();
+    QVERIFY(QTest::qWaitForWindowExposed(rhiWidget));
+    QTRY_VERIFY(frameSpy.count() > 0);
+    QCOMPARE(errorSpy.count(), 0);
+
+    frameSpy.clear();
+    rhiWidget->setParent(windowOne);
     windowOne->show();
     QVERIFY(QTest::qWaitForWindowExposed(windowOne));
     QTRY_VERIFY(frameSpy.count() > 0);