diff options
author | Friedemann Kleint <Friedemann.Kleint@nokia.com> | 2012-07-18 13:12:59 +0200 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-07-20 09:27:28 +0200 |
commit | 87618403976aa82450895da6196a944195980de7 (patch) | |
tree | 906d035678d8a31e57a94c01e8756b0085b3e1e4 /tests/auto/widgets/widgets | |
parent | c081107206cb1d31415539c4429f7896b997cf71 (diff) |
Implement waitForWindowExposed and friends for widget windows.
- Implement waitForWindowExposed() for toplevel windows.
- Implement waitForWindowShown(QWidget *) and mark as
deprecated in line with waitForWindowShown(QWindow*).
- Use in tests.
- Simplify tests (collapse waitForExposed, setActive
into setActiveWindow, waitForActive), remove most
hard-coded timeouts.
- Stabilize graphicsview tests by using waitForWindowActive.
Change-Id: Ic7c061e2745b36f71a715ee4e47c0346b11a91e8
Reviewed-by: Jason McDonald <jason.mcdonald@nokia.com>
Diffstat (limited to 'tests/auto/widgets/widgets')
14 files changed, 79 insertions, 209 deletions
diff --git a/tests/auto/widgets/widgets/qcombobox/tst_qcombobox.cpp b/tests/auto/widgets/widgets/qcombobox/tst_qcombobox.cpp index ce0049cc2d..96dce56b8b 100644 --- a/tests/auto/widgets/widgets/qcombobox/tst_qcombobox.cpp +++ b/tests/auto/widgets/widgets/qcombobox/tst_qcombobox.cpp @@ -2095,11 +2095,8 @@ void tst_QComboBox::task190205_setModelAdjustToContents() correctBox.addItems(finalContent); correctBox.show(); - QCoreApplication::processEvents(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&box); - qt_x11_wait_for_window_manager(&correctBox); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&box)); + QVERIFY(QTest::qWaitForWindowExposed(&correctBox)); // box should be resized to the same size as correctBox QTRY_COMPARE(box.size(), correctBox.size()); diff --git a/tests/auto/widgets/widgets/qdockwidget/tst_qdockwidget.cpp b/tests/auto/widgets/widgets/qdockwidget/tst_qdockwidget.cpp index 43c0022c25..494ba548c1 100644 --- a/tests/auto/widgets/widgets/qdockwidget/tst_qdockwidget.cpp +++ b/tests/auto/widgets/widgets/qdockwidget/tst_qdockwidget.cpp @@ -352,9 +352,7 @@ void tst_QDockWidget::setFloating() mw.addDockWidget(Qt::LeftDockWidgetArea, &dw); mw.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mw); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mw)); QVERIFY(!dw.isFloating()); @@ -719,9 +717,7 @@ void tst_QDockWidget::task165177_deleteFocusWidget() QLineEdit *ledit = new QLineEdit; dw->setWidget(ledit); mw.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mw); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mw)); qApp->processEvents(); dw->setFloating(true); delete ledit; @@ -760,19 +756,15 @@ void tst_QDockWidget::task169808_setFloating() mw.addDockWidget(Qt::LeftDockWidgetArea, dw); dw->setFloating(true); mw.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mw); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mw)); QCOMPARE(dw->widget()->size(), dw->widget()->sizeHint()); //and now we try to test if the contents margin is taken into account dw->widget()->setSizePolicy(QSizePolicy::Fixed, QSizePolicy::Fixed); dw->setFloating(false); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mw); -#endif - QTest::qWait(100); //leave time processing events + QVERIFY(QTest::qWaitForWindowExposed(&mw)); + qApp->processEvents(); //leave time processing events const QSize oldSize = dw->size(); @@ -780,10 +772,8 @@ void tst_QDockWidget::task169808_setFloating() dw->setContentsMargins(margin, margin, margin, margin); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mw); -#endif - QTest::qWait(100); //leave time processing events + QVERIFY(QTest::qWaitForWindowExposed(&mw)); + qApp->processEvents(); //leave time processing events //widget size shouldn't have changed QCOMPARE(dw->widget()->size(), dw->widget()->sizeHint()); diff --git a/tests/auto/widgets/widgets/qdoublespinbox/tst_qdoublespinbox.cpp b/tests/auto/widgets/widgets/qdoublespinbox/tst_qdoublespinbox.cpp index f140e631f2..45b9653982 100644 --- a/tests/auto/widgets/widgets/qdoublespinbox/tst_qdoublespinbox.cpp +++ b/tests/auto/widgets/widgets/qdoublespinbox/tst_qdoublespinbox.cpp @@ -1066,9 +1066,9 @@ void tst_QDoubleSpinBox::taskQTBUG_5008_textFromValueAndValidate() spinbox.activateWindow(); spinbox.setFocus(); QApplication::setActiveWindow(&spinbox); - QTest::qWaitForWindowShown(&spinbox); + QVERIFY(QTest::qWaitForWindowActive(&spinbox)); + QCOMPARE(static_cast<QWidget *>(&spinbox), QApplication::activeWindow()); QTRY_VERIFY(spinbox.hasFocus()); - QTRY_COMPARE(static_cast<QWidget *>(&spinbox), QApplication::activeWindow()); QCOMPARE(spinbox.text(), spinbox.locale().toString(spinbox.value())); spinbox.lineEdit()->setCursorPosition(2); //just after the first thousand separator QTest::keyClick(static_cast<QWidget *>(0), Qt::Key_0); // let's insert a 0 diff --git a/tests/auto/widgets/widgets/qgroupbox/tst_qgroupbox.cpp b/tests/auto/widgets/widgets/qgroupbox/tst_qgroupbox.cpp index 730992da77..ab382236d8 100644 --- a/tests/auto/widgets/widgets/qgroupbox/tst_qgroupbox.cpp +++ b/tests/auto/widgets/widgets/qgroupbox/tst_qgroupbox.cpp @@ -464,8 +464,8 @@ void tst_QGroupBox::propagateFocus() QLineEdit lineEdit(&box); box.show(); QApplication::setActiveWindow(&box); + QVERIFY(QTest::qWaitForWindowActive(&box)); box.setFocus(); - QTest::qWait(250); QTRY_COMPARE(qApp->focusWidget(), static_cast<QWidget*>(&lineEdit)); } diff --git a/tests/auto/widgets/widgets/qlineedit/tst_qlineedit.cpp b/tests/auto/widgets/widgets/qlineedit/tst_qlineedit.cpp index 35b90046e3..48ff37a448 100644 --- a/tests/auto/widgets/widgets/qlineedit/tst_qlineedit.cpp +++ b/tests/auto/widgets/widgets/qlineedit/tst_qlineedit.cpp @@ -3338,8 +3338,8 @@ void tst_QLineEdit::task174640_editingFinished() mw.show(); QApplication::setActiveWindow(&mw); mw.activateWindow(); - QTest::qWaitForWindowShown(&mw); - QTRY_COMPARE(&mw, QApplication::activeWindow()); + QVERIFY(QTest::qWaitForWindowActive(&mw)); + QCOMPARE(&mw, QApplication::activeWindow()); QSignalSpy editingFinishedSpy(le1, SIGNAL(editingFinished())); @@ -3448,10 +3448,7 @@ void tst_QLineEdit::task229938_dontEmitChangedWhenTextIsNotChanged() QLineEdit lineEdit; lineEdit.setMaxLength(5); lineEdit.show(); -#ifdef Q_WS_X11 - // to be safe and avoid failing setFocus with window managers - qt_x11_wait_for_window_manager(&lineEdit); -#endif + QTest::qWaitForWindowExposed(&lineEdit); // to be safe and avoid failing setFocus with window managers lineEdit.setFocus(); QSignalSpy changedSpy(&lineEdit, SIGNAL(textChanged(QString))); QTest::qWait(200); @@ -3648,8 +3645,8 @@ void tst_QLineEdit::taskQTBUG_7395_readOnlyShortcut() le.addAction(&action); le.show(); - QTest::qWaitForWindowShown(&le); QApplication::setActiveWindow(&le); + QVERIFY(QTest::qWaitForWindowActive(&le)); le.setFocus(); QTRY_VERIFY(le.hasFocus()); diff --git a/tests/auto/widgets/widgets/qmdiarea/tst_qmdiarea.cpp b/tests/auto/widgets/widgets/qmdiarea/tst_qmdiarea.cpp index 7aa9bb2ddf..4a0478394d 100644 --- a/tests/auto/widgets/widgets/qmdiarea/tst_qmdiarea.cpp +++ b/tests/auto/widgets/widgets/qmdiarea/tst_qmdiarea.cpp @@ -464,12 +464,8 @@ void tst_QMdiArea::subWindowActivated2() mdiArea.addSubWindow(new QWidget); QCOMPARE(spy.count(), 0); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif - QTest::qWaitForWindowShown(&mdiArea); mdiArea.activateWindow(); - QTest::qWait(100); + QVERIFY(QTest::qWaitForWindowActive(&mdiArea)); QTRY_COMPARE(spy.count(), 5); QCOMPARE(mdiArea.activeSubWindow(), mdiArea.subWindowList().back()); @@ -503,10 +499,7 @@ void tst_QMdiArea::subWindowActivated2() spy.clear(); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif - QTest::qWait(100); + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); QTRY_COMPARE(spy.count(), 1); QCOMPARE(mdiArea.activeSubWindow(), activeSubWindow); spy.clear(); @@ -517,13 +510,11 @@ void tst_QMdiArea::subWindowActivated2() // Check that we only emit _one_ signal and the active window // is unchanged after showMinimized/showNormal. mdiArea.showMinimized(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#elif defined (Q_OS_MAC) +#if defined (Q_OS_MAC) if (!macHasAccessToWindowsServer()) QEXPECT_FAIL("", "showMinimized doesn't really minimize if you don't have access to the server", Abort); #endif - QTest::qWait(10); + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); #if defined(Q_WS_QWS) QEXPECT_FAIL("", "task 168682", Abort); #endif @@ -538,11 +529,11 @@ void tst_QMdiArea::subWindowActivated2() QCOMPARE(mdiArea.currentSubWindow(), activeSubWindow); spy.clear(); + // For this test, the QMdiArea widget must be active after minimizing and + // showing it again. QMdiArea has no active sub window if it is inactive itself. + qApp->setActiveWindow(&mdiArea); mdiArea.showNormal(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif - QTest::qWait(100); + QVERIFY(QTest::qWaitForWindowActive(&mdiArea)); QTRY_COMPARE(spy.count(), 1); QCOMPARE(mdiArea.activeSubWindow(), activeSubWindow); spy.clear(); @@ -998,11 +989,8 @@ void tst_QMdiArea::activeSubWindow() mainWindow.addDockWidget(Qt::LeftDockWidgetArea, dockWidget); mainWindow.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mainWindow); -#endif - qApp->setActiveWindow(&mainWindow); + QVERIFY(QTest::qWaitForWindowActive(&mainWindow)); QCOMPARE(mdiArea->activeSubWindow(), subWindow); QCOMPARE(qApp->focusWidget(), (QWidget *)subWindowLineEdit); @@ -1022,9 +1010,7 @@ void tst_QMdiArea::activeSubWindow() QLineEdit dummyTopLevel; dummyTopLevel.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&dummyTopLevel); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&dummyTopLevel)); qApp->setActiveWindow(&dummyTopLevel); QCOMPARE(mdiArea->activeSubWindow(), subWindow); @@ -1047,9 +1033,7 @@ void tst_QMdiArea::currentSubWindow() { QMdiArea mdiArea; mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); for (int i = 0; i < 5; ++i) mdiArea.addSubWindow(new QLineEdit)->show(); @@ -1065,9 +1049,7 @@ void tst_QMdiArea::currentSubWindow() QLineEdit dummyTopLevel; dummyTopLevel.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&dummyTopLevel); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&dummyTopLevel)); // Move focus to another top-level and check that we still // have an active window. @@ -1130,9 +1112,7 @@ void tst_QMdiArea::addAndRemoveWindows() QMdiArea workspace(&topLevel); workspace.resize(800, 600); topLevel.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&workspace); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&topLevel)); { // addSubWindow with large widget QCOMPARE(workspace.subWindowList().count(), 0); @@ -1605,9 +1585,7 @@ void tst_QMdiArea::tileSubWindows() if (PlatformQuirks::isAutoMaximizing()) workspace.setWindowFlags(workspace.windowFlags() | Qt::X11BypassWindowManagerHint); workspace.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&workspace); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&workspace)); const int windowCount = 10; for (int i = 0; i < windowCount; ++i) { @@ -1784,9 +1762,7 @@ void tst_QMdiArea::tileSubWindows() QCOMPARE(vBar->minimum(), 0); workspace.tileSubWindows(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&workspace); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&workspace)); qApp->processEvents(); QTRY_VERIFY(workspace.size() != QSize(150, 150)); @@ -1799,9 +1775,7 @@ void tst_QMdiArea::cascadeAndTileSubWindows() QMdiArea workspace; workspace.resize(400, 400); workspace.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&workspace); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&workspace)); const int windowCount = 10; QList<QMdiSubWindow *> windows; @@ -1867,10 +1841,7 @@ void tst_QMdiArea::resizeMaximizedChildWindows() QWidget topLevel; QMdiArea workspace(&topLevel); topLevel.show(); -#if defined(Q_WS_X11) - qt_x11_wait_for_window_manager(&workspace); -#endif - QTest::qWait(100); + QVERIFY(QTest::qWaitForWindowExposed(&topLevel)); workspace.resize(startSize, startSize); workspace.setOption(QMdiArea::DontMaximizeSubWindowOnActivation); QSize workspaceSize = workspace.size(); @@ -1932,9 +1903,7 @@ void tst_QMdiArea::dontMaximizeSubWindowOnActivation() { QMdiArea mdiArea; mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); qApp->setActiveWindow(&mdiArea); // Add one maximized window. @@ -2040,9 +2009,7 @@ void tst_QMdiArea::delayedPlacement() mdiArea.resize(window3->minimumSizeHint().width() * 3, 400); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); QCOMPARE(window1->geometry().topLeft(), QPoint(0, 0)); QCOMPARE(window2->geometry().topLeft(), window1->geometry().topRight() + QPoint(1, 0)); @@ -2058,9 +2025,7 @@ void tst_QMdiArea::iconGeometryInMenuBar() QMdiSubWindow *subWindow = mdiArea->addSubWindow(new QWidget); mainWindow.setCentralWidget(mdiArea); mainWindow.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mainWindow); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mainWindow)); subWindow->showMaximized(); QVERIFY(subWindow->isMaximized()); @@ -2108,10 +2073,7 @@ void tst_QMdiArea::resizeTimer() QMdiArea mdiArea; QMdiSubWindow *subWindow = mdiArea.addSubWindow(new QWidget); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif - QTest::qWaitForWindowShown(&mdiArea); + QVERIFY(QTest::qWaitForWindowActive(&mdiArea)); #ifndef Q_OS_WINCE int time = 250; @@ -2119,8 +2081,6 @@ void tst_QMdiArea::resizeTimer() int time = 1000; #endif - QTest::qWait(time); - EventSpy timerEventSpy(subWindow, QEvent::Timer); QCOMPARE(timerEventSpy.count(), 0); @@ -2149,9 +2109,7 @@ void tst_QMdiArea::updateScrollBars() QMdiSubWindow *subWindow2 = mdiArea.addSubWindow(new QWidget); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); qApp->processEvents(); QScrollBar *hbar = mdiArea.horizontalScrollBar(); @@ -2178,9 +2136,7 @@ void tst_QMdiArea::updateScrollBars() // We still shouldn't get any scroll bars. mdiArea.resize(mdiArea.size() - QSize(20, 20)); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); qApp->processEvents(); QVERIFY(subWindow1->isMaximized()); QVERIFY(!hbar->isVisible()); @@ -2272,9 +2228,7 @@ void tst_QMdiArea::setActivationOrder() QCOMPARE(mdiArea.subWindowList(activationOrder), subWindows); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); for (int i = 0; i < subWindows.count(); ++i) { mdiArea.activateNextSubWindow(); @@ -2333,9 +2287,7 @@ void tst_QMdiArea::tabBetweenSubWindows() subWindows << mdiArea.addSubWindow(new QLineEdit); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); qApp->setActiveWindow(&mdiArea); QWidget *focusWidget = subWindows.back()->widget(); @@ -2389,9 +2341,7 @@ void tst_QMdiArea::setViewMode() } mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); QMdiSubWindow *activeSubWindow = mdiArea.activeSubWindow(); const QList<QMdiSubWindow *> subWindows = mdiArea.subWindowList(); @@ -2553,9 +2503,7 @@ void tst_QMdiArea::setTabShape() QMdiArea mdiArea; mdiArea.addSubWindow(new QWidget); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); // Default. QCOMPARE(mdiArea.tabShape(), QTabWidget::Rounded); @@ -2601,9 +2549,7 @@ void tst_QMdiArea::setTabPosition() QMdiArea mdiArea; mdiArea.addSubWindow(new QWidget); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); // Make sure there are no margins. mdiArea.setContentsMargins(0, 0, 0, 0); @@ -2661,9 +2607,7 @@ void tst_QMdiArea::nativeSubWindows() mdiArea.addSubWindow(new QWidget); mdiArea.addSubWindow(new QWidget); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); // No native widgets. QVERIFY(!mdiArea.viewport()->internalWinId()); @@ -2694,9 +2638,7 @@ void tst_QMdiArea::nativeSubWindows() (void)nativeWidget->winId(); mdiArea.addSubWindow(nativeWidget); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); // The viewport and all the sub-windows must be native. QVERIFY(mdiArea.viewport()->internalWinId()); @@ -2709,9 +2651,7 @@ void tst_QMdiArea::nativeSubWindows() mdiArea.addSubWindow(new QWidget); mdiArea.addSubWindow(new QWidget); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); QMdiSubWindow *nativeSubWindow = mdiArea.subWindowList().last(); QVERIFY(!nativeSubWindow->internalWinId()); @@ -2734,9 +2674,7 @@ void tst_QMdiArea::nativeSubWindows() mdiArea.addSubWindow(new QWidget); mdiArea.addSubWindow(new QWidget); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); const QGLContext *context = glViewport->context(); if (!context || !context->isValid()) diff --git a/tests/auto/widgets/widgets/qmdisubwindow/tst_qmdisubwindow.cpp b/tests/auto/widgets/widgets/qmdisubwindow/tst_qmdisubwindow.cpp index 79afd14481..98a3675422 100644 --- a/tests/auto/widgets/widgets/qmdisubwindow/tst_qmdisubwindow.cpp +++ b/tests/auto/widgets/widgets/qmdisubwindow/tst_qmdisubwindow.cpp @@ -64,9 +64,6 @@ #endif QT_BEGIN_NAMESPACE -#if defined(Q_WS_X11) -extern void qt_x11_wait_for_window_manager(QWidget *w); -#endif #if !defined(Q_WS_WIN) extern bool qt_tab_all_widgets; #endif @@ -254,9 +251,7 @@ void tst_QMdiSubWindow::minimumSize() QMdiSubWindow *subWindow1 = mdiArea.addSubWindow(new QWidget); subWindow1->setMinimumSize(1000, 1000); mdiArea.show(); -#if defined(Q_WS_X11) - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); QCOMPARE(subWindow1->size(), QSize(1000, 1000)); // Check that we respect the minimum size set on the internal widget. @@ -322,9 +317,7 @@ void tst_QMdiSubWindow::setWindowState() QMdiSubWindow *window = qobject_cast<QMdiSubWindow *>(workspace.addSubWindow(new QLineEdit)); window->show(); workspace.show(); -#if defined(Q_WS_X11) - qt_x11_wait_for_window_manager(&workspace); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&workspace)); QWidget *testWidget = 0; for (int iteration = 0; iteration < 2; ++iteration) { @@ -535,9 +528,7 @@ void tst_QMdiSubWindow::emittingOfSignals() window->setParent(0); window->showNormal(); -#if defined(Q_WS_X11) - qt_x11_wait_for_window_manager(window); -#endif + QVERIFY(QTest::qWaitForWindowExposed(window)); qApp->processEvents(); spy.clear(); @@ -555,9 +546,7 @@ void tst_QMdiSubWindow::showShaded() window->resize(300, 300); qApp->processEvents(); workspace.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&workspace); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&workspace)); QVERIFY(!window->isShaded()); QVERIFY(!window->isMaximized()); @@ -652,9 +641,7 @@ void tst_QMdiSubWindow::showNormal() qApp->processEvents(); workspace.show(); window->show(); -#if defined(Q_WS_X11) - qt_x11_wait_for_window_manager(&workspace); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&workspace)); QRect originalGeometry = window->geometry(); QVERIFY(QMetaObject::invokeMethod(window, slot.data())); @@ -718,9 +705,7 @@ void tst_QMdiSubWindow::setOpaqueResizeAndMove() qApp->processEvents(); workspace.resize(workspaceSize); workspace.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&workspace); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&workspace)); QWidget *mouseReceiver = 0; if (window->style()->inherits("QMacStyle")) @@ -918,9 +903,7 @@ void tst_QMdiSubWindow::setWindowFlags() qApp->processEvents(); workspace.show(); window->show(); -#if defined(Q_WS_X11) - qt_x11_wait_for_window_manager(&workspace); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&workspace)); window->setWindowFlags(windowType | customFlags); QCOMPARE(window->windowType(), expectedWindowType); @@ -1387,9 +1370,7 @@ void tst_QMdiSubWindow::resizeEvents() QMdiArea *mdiArea = new QMdiArea; mainWindow.setCentralWidget(mdiArea); mainWindow.show(); -#if defined(Q_WS_X11) - qt_x11_wait_for_window_manager(&mainWindow); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mainWindow)); QMdiSubWindow *window = mdiArea->addSubWindow(new QTextEdit); window->show(); @@ -1464,9 +1445,7 @@ void tst_QMdiSubWindow::hideAndShow() QMenuBar *menuBar = mainWindow.menuBar(); mainWindow.setCentralWidget(tabWidget); mainWindow.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mainWindow); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mainWindow)); QVERIFY(!menuBar->cornerWidget(Qt::TopRightCorner)); QMdiSubWindow *subWindow = mdiArea->addSubWindow(new QTextEdit); @@ -1559,9 +1538,7 @@ void tst_QMdiSubWindow::keepWindowMaximizedState() QMdiArea mdiArea; QMdiSubWindow *subWindow = mdiArea.addSubWindow(new QTextEdit); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); subWindow->showMaximized(); QVERIFY(subWindow->isMaximized()); @@ -1600,9 +1577,7 @@ void tst_QMdiSubWindow::explicitlyHiddenWidget() textEdit->hide(); QMdiSubWindow *subWindow = mdiArea.addSubWindow(textEdit); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); QVERIFY(subWindow->isVisible()); QVERIFY(!textEdit->isVisible()); @@ -1673,9 +1648,7 @@ void tst_QMdiSubWindow::fixedMinMaxSize() QMdiArea mdiArea; mdiArea.setGeometry(0, 0, 640, 480); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); const QSize minimumSize = QSize(250, 150); const QSize maximumSize = QSize(300, 200); @@ -1744,9 +1717,7 @@ void tst_QMdiSubWindow::replaceMenuBarWhileMaximized() mainWindow.setCentralWidget(mdiArea); QMenuBar *menuBar = mainWindow.menuBar(); mainWindow.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mainWindow); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mainWindow)); qApp->processEvents(); @@ -1803,9 +1774,7 @@ void tst_QMdiSubWindow::closeOnDoubleClick() QMdiArea mdiArea; QPointer<QMdiSubWindow> subWindow = mdiArea.addSubWindow(new QWidget); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); subWindow->showSystemMenu(); QTest::qWait(200); @@ -1831,9 +1800,8 @@ void tst_QMdiSubWindow::setFont() subWindow->resize(300, 100); subWindow->setWindowTitle(QLatin1String("Window title")); mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); + const QFont originalFont = QApplication::font("QMdiSubWindowTitleBar"); QStyleOptionTitleBar opt; @@ -1870,9 +1838,7 @@ void tst_QMdiSubWindow::task_188849() QMdiSubWindow *subWindow = mdiArea->addSubWindow(new QWidget); mainWindow.setCentralWidget(mdiArea); mainWindow.show(); -#if defined(Q_WS_X11) - qt_x11_wait_for_window_manager(&mainWindow); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mainWindow)); // QMdiSubWindow will now try to show its buttons in the menu bar. // Without checking that the menu bar is actually a QMenuBar @@ -2015,9 +1981,7 @@ void tst_QMdiSubWindow::task_226929() { QMdiArea mdiArea; mdiArea.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mdiArea); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mdiArea)); QMdiSubWindow *sub1 = mdiArea.addSubWindow(new QTextEdit); sub1->showMinimized(); diff --git a/tests/auto/widgets/widgets/qmenu/tst_qmenu.cpp b/tests/auto/widgets/widgets/qmenu/tst_qmenu.cpp index d5e8dce860..89437354df 100644 --- a/tests/auto/widgets/widgets/qmenu/tst_qmenu.cpp +++ b/tests/auto/widgets/widgets/qmenu/tst_qmenu.cpp @@ -553,8 +553,8 @@ void tst_QMenu::tearOff() QVERIFY(menu->isTearOffEnabled()); widget.show(); - QTest::qWaitForWindowShown(&widget); widget.activateWindow(); + QVERIFY(QTest::qWaitForWindowActive(&widget)); menu->popup(QPoint(0,0)); QTest::qWait(50); QVERIFY(!menu->isTearOffMenuVisible()); diff --git a/tests/auto/widgets/widgets/qmenubar/tst_qmenubar.cpp b/tests/auto/widgets/widgets/qmenubar/tst_qmenubar.cpp index 6f7d7517e8..5a2cf0f316 100644 --- a/tests/auto/widgets/widgets/qmenubar/tst_qmenubar.cpp +++ b/tests/auto/widgets/widgets/qmenubar/tst_qmenubar.cpp @@ -1264,8 +1264,8 @@ void tst_QMenuBar::taskQTBUG11823_crashwithInvisibleActions() QAction * a = menubar.addAction( "&a" ); menubar.show(); - QTest::qWaitForWindowShown(&menubar); QApplication::setActiveWindow(&menubar); + QVERIFY(QTest::qWaitForWindowActive(&menubar)); menubar.setActiveAction(m); QCOMPARE(menubar.activeAction(), m); QTest::keyClick(static_cast<QWidget *>(0), Qt::Key_Right); diff --git a/tests/auto/widgets/widgets/qprogressbar/tst_qprogressbar.cpp b/tests/auto/widgets/widgets/qprogressbar/tst_qprogressbar.cpp index 89f2508a05..84e3acb346 100644 --- a/tests/auto/widgets/widgets/qprogressbar/tst_qprogressbar.cpp +++ b/tests/auto/widgets/widgets/qprogressbar/tst_qprogressbar.cpp @@ -219,9 +219,7 @@ void tst_QProgressBar::setMinMaxRepaint() pbar.setMaximum(10); pbar.setFormat("%v"); pbar.show(); - QTest::qWaitForWindowShown(&pbar); - - QApplication::processEvents(); + QVERIFY(QTest::qWaitForWindowActive(&pbar)); // No repaint when setting minimum to the current minimum pbar.repainted = false; diff --git a/tests/auto/widgets/widgets/qsizegrip/tst_qsizegrip.cpp b/tests/auto/widgets/widgets/qsizegrip/tst_qsizegrip.cpp index a0a314bd9d..c40ad189c6 100644 --- a/tests/auto/widgets/widgets/qsizegrip/tst_qsizegrip.cpp +++ b/tests/auto/widgets/widgets/qsizegrip/tst_qsizegrip.cpp @@ -88,11 +88,8 @@ public: void changeEvent(QEvent *event) { QWidget::changeEvent(event); - if (isWindow() && event->type() == QEvent::WindowStateChange) { -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(this); -#endif - } + if (isWindow() && event->type() == QEvent::WindowStateChange) + QVERIFY(QTest::qWaitForWindowExposed(this)); } }; diff --git a/tests/auto/widgets/widgets/qspinbox/tst_qspinbox.cpp b/tests/auto/widgets/widgets/qspinbox/tst_qspinbox.cpp index 5105ed67ba..13da4430c2 100644 --- a/tests/auto/widgets/widgets/qspinbox/tst_qspinbox.cpp +++ b/tests/auto/widgets/widgets/qspinbox/tst_qspinbox.cpp @@ -919,13 +919,10 @@ void tst_QSpinBox::specialValue() spin.setMaximum(100); spin.setValue(50); topWidget.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&topWidget); -#endif - QTest::qWait(100); //make sure we have the focus (even if editingFinished fails) qApp->setActiveWindow(&topWidget); topWidget.activateWindow(); + QVERIFY(QTest::qWaitForWindowActive(&topWidget)); spin.setFocus(); QTest::keyClick(&spin, Qt::Key_Return); diff --git a/tests/auto/widgets/widgets/qstatusbar/tst_qstatusbar.cpp b/tests/auto/widgets/widgets/qstatusbar/tst_qstatusbar.cpp index ca54564d24..1f7e746894 100644 --- a/tests/auto/widgets/widgets/qstatusbar/tst_qstatusbar.cpp +++ b/tests/auto/widgets/widgets/qstatusbar/tst_qstatusbar.cpp @@ -172,9 +172,7 @@ void tst_QStatusBar::setSizeGripEnabled() QPointer<QStatusBar> statusBar = mainWindow.statusBar(); QVERIFY(statusBar); mainWindow.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mainWindow); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mainWindow)); QTRY_VERIFY(statusBar->isVisible()); QPointer<QSizeGrip> sizeGrip = qFindChild<QSizeGrip *>(statusBar); @@ -203,9 +201,7 @@ void tst_QStatusBar::setSizeGripEnabled() #ifndef Q_OS_MAC // Work around Lion fullscreen issues on CI system - QTQAINFRA-506 mainWindow.showFullScreen(); #endif -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&mainWindow); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&mainWindow)); qApp->processEvents(); mainWindow.setStatusBar(new QStatusBar(&mainWindow)); @@ -265,15 +261,15 @@ void tst_QStatusBar::QTBUG4334_hiddenOnMaximizedWindow() statusbar.setSizeGripEnabled(true); main.setStatusBar(&statusbar); main.showMaximized(); - QTest::qWaitForWindowShown(&main); + QVERIFY(QTest::qWaitForWindowActive(&main)); #ifndef Q_OS_MAC QVERIFY(!statusbar.findChild<QSizeGrip*>()->isVisible()); #endif main.showNormal(); - QTest::qWaitForWindowShown(&main); + QVERIFY(QTest::qWaitForWindowExposed(&main)); QVERIFY(statusbar.findChild<QSizeGrip*>()->isVisible()); main.showFullScreen(); - QTest::qWaitForWindowShown(&main); + QVERIFY(QTest::qWaitForWindowExposed(&main)); QVERIFY(!statusbar.findChild<QSizeGrip*>()->isVisible()); } diff --git a/tests/auto/widgets/widgets/qtoolbar/tst_qtoolbar.cpp b/tests/auto/widgets/widgets/qtoolbar/tst_qtoolbar.cpp index f4fb6a9bcd..a7db9a01f7 100644 --- a/tests/auto/widgets/widgets/qtoolbar/tst_qtoolbar.cpp +++ b/tests/auto/widgets/widgets/qtoolbar/tst_qtoolbar.cpp @@ -529,9 +529,7 @@ void tst_QToolBar::actionGeometry() tb.addAction(&action4); tb.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&tb); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&tb)); QList<QToolBarExtension *> extensions = tb.findChildren<QToolBarExtension *>(); @@ -853,9 +851,7 @@ void tst_QToolBar::actionTriggered() tb.addAction(&action4); tb.show(); -#ifdef Q_WS_X11 - qt_x11_wait_for_window_manager(&tb); -#endif + QVERIFY(QTest::qWaitForWindowExposed(&tb)); QList<QToolBarExtension *> extensions = tb.findChildren<QToolBarExtension *>(); |