diff options
author | Gunnar Sletta <gunnar.sletta@digia.com> | 2013-10-17 14:53:33 +0200 |
---|---|---|
committer | The Qt Project <gerrit-noreply@qt-project.org> | 2013-10-30 08:29:49 +0100 |
commit | 906d5c5c40183468f9521277c6244a6c46730de6 (patch) | |
tree | 0eb46a8f88d59993ab659e2dc07970d1ce2f0d73 /tests/auto/quick/qquickwindow/tst_qquickwindow.cpp | |
parent | c084d32d92b2df55532fa1599e590c29bf2b5bfb (diff) |
Use one render loop per QQuickWindow
See the task for the full reasoning behind this patch.
The threaded renderloop has been refactored to have one window per
thread. This is mostly a simplification of the current code path where
for loops over multiple windows are turned into if (window).
The QSGContext has been split into two classes, QSGRenderContext
for which there is one per OpenGLContext.
The rest of the patch is name changes and a couple of cleanups
in the hopes of simplifying this change.
Task-number: QTBUG-33993
Change-Id: I31c81f9694d7da7474a72333169be38de62613c4
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Diffstat (limited to 'tests/auto/quick/qquickwindow/tst_qquickwindow.cpp')
-rw-r--r-- | tests/auto/quick/qquickwindow/tst_qquickwindow.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/tests/auto/quick/qquickwindow/tst_qquickwindow.cpp b/tests/auto/quick/qquickwindow/tst_qquickwindow.cpp index 28dacb1f81..984881c8da 100644 --- a/tests/auto/quick/qquickwindow/tst_qquickwindow.cpp +++ b/tests/auto/quick/qquickwindow/tst_qquickwindow.cpp @@ -1093,7 +1093,11 @@ void tst_qquickwindow::noUpdateWhenNothingChanges() QQuickRectangle rect(window.contentItem()); window.showNormal(); - QTRY_VERIFY(window.isExposed()); + QTest::qWaitForWindowExposed(&window); + // Many platforms are broken in the sense that that they follow up + // the initial expose with a second expose or more. Let these go + // through before we let the test continue. + QTest::qWait(100); if (window.openglContext()->thread() == QGuiApplication::instance()->thread()) { QSKIP("Only threaded renderloop implements this feature"); @@ -1102,7 +1106,8 @@ void tst_qquickwindow::noUpdateWhenNothingChanges() QSignalSpy spy(&window, SIGNAL(frameSwapped())); rect.update(); - QTest::qWait(500); + // Wait a while and verify that no more frameSwapped come our way. + QTest::qWait(100); QCOMPARE(spy.size(), 0); } @@ -1348,11 +1353,6 @@ void tst_qquickwindow::hideThenDelete_data() void tst_qquickwindow::hideThenDelete() { - if (QGuiApplication::platformName() == QStringLiteral("xcb")) { - QSKIP("For some obscure reason this test fails in CI only"); - return; - } - QFETCH(bool, persistentSG); QFETCH(bool, persistentGL); |