diff options
| author | Peter Varga <pvarga@inf.u-szeged.hu> | 2019-10-09 08:38:25 +0200 |
|---|---|---|
| committer | Peter Varga <pvarga@inf.u-szeged.hu> | 2019-10-09 17:41:00 +0200 |
| commit | 120a5dd51e07f2de733e8d8b305897160851df19 (patch) | |
| tree | 868c7667c895c88058314377fe8400bb01be505c /tests | |
| parent | b44ed0fb99cd3d7047fcd322010c27cfa7989a18 (diff) | |
Stabilize tst_QWebEnginePage::comboBoxPopupPositionAfterMove auto test
Fixes a flaky fail when jsViewPosition is called too early to query the
view position.
Change-Id: Ib2e9c1e95a6363a8e45e8bcc44e1878fd9a6914f
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp b/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp index 6373bf2d9..818439e46 100644 --- a/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp +++ b/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp @@ -1051,6 +1051,12 @@ void tst_QWebEnginePage::comboBoxPopupPositionAfterMove() auto jsViewPosition = [&view]() { QLatin1String script("(function() { return [window.screenX, window.screenY]; })()"); QVariantList posList = evaluateJavaScriptSync(view.page(), script).toList(); + + if (posList.count() != 2) { + qWarning("jsViewPosition failed."); + return QPoint(); + } + return QPoint(posList.at(0).toInt(), posList.at(1).toInt()); }; |