From 0aded4662dfab4b213b81e08998b2538a8c8dd0f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Michael=20Br=C3=BCning?= Date: Tue, 16 May 2023 18:30:06 +0200 Subject: Assert that the primary screen is available before using it MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Does not fix the underlying issue, but may give us more meaningful information about the seemingly random crashes we have seen on the CI. Change-Id: I239fd05bf6a7add38e17830aad59f961036871d9 Reviewed-by: Michael Brüning --- tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp b/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp index ba88fd99e..9c6a7ca2d 100644 --- a/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp +++ b/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp @@ -1359,6 +1359,7 @@ void tst_QWebEnginePage::comboBoxPopupPositionAfterMove_data() void tst_QWebEnginePage::comboBoxPopupPositionAfterMove() { QWebEngineView view; + Q_ASSERT(QGuiApplication::primaryScreen()); view.move(QGuiApplication::primaryScreen()->availableGeometry().topLeft()); view.resize(640, 480); view.show(); @@ -1430,6 +1431,7 @@ void tst_QWebEnginePage::comboBoxPopupPositionAfterChildMove() mainWidget.layout()->addWidget(&view); QScreen *screen = QGuiApplication::primaryScreen(); + Q_ASSERT(screen); mainWidget.move(screen->availableGeometry().topLeft()); mainWidget.resize(640, 480); mainWidget.show(); -- cgit v1.2.1