From 4354fef3d4c45a9ac1c01cf99627aad6d3b4d68d Mon Sep 17 00:00:00 2001 From: Friedemann Kleint Date: Thu, 28 Aug 2014 12:59:14 +0200 Subject: Windows: Prevent hidden transient children from being re-shown by Windows. Bring back code from Qt 4 to handle WM_SHOWWINDOW / SW_PARENTOPENING correctly. Task-number: QTBUG-40696 Change-Id: If018bf90573f495dbe32d0c46f522ccde0691ebb Reviewed-by: Laszlo Agocs --- tests/auto/gui/kernel/qwindow/tst_qwindow.cpp | 59 ++++++++++++++++++++++++++- 1 file changed, 58 insertions(+), 1 deletion(-) (limited to 'tests') diff --git a/tests/auto/gui/kernel/qwindow/tst_qwindow.cpp b/tests/auto/gui/kernel/qwindow/tst_qwindow.cpp index 0686ca792c..25e5255189 100644 --- a/tests/auto/gui/kernel/qwindow/tst_qwindow.cpp +++ b/tests/auto/gui/kernel/qwindow/tst_qwindow.cpp @@ -39,10 +39,11 @@ ** ****************************************************************************/ -#include +#include #include #include #include +#include #include @@ -51,6 +52,8 @@ #if defined(Q_OS_QNX) #include +#elif defined(Q_OS_WIN) && !defined(Q_OS_WINCE) && !defined(Q_OS_WINRT) +# include #endif // For QSignalSpy slot connections. @@ -94,6 +97,7 @@ private slots: void modalWithChildWindow(); void modalWindowModallity(); void modalWindowPosition(); + void windowsTransientChildren(); void initTestCase(); void cleanup(); @@ -1539,6 +1543,59 @@ void tst_QWindow::modalWindowPosition() QCOMPARE(window.geometry(), origGeo); } +class ColoredWindow : public QRasterWindow { +public: + explicit ColoredWindow(const QColor &color, QWindow *parent = 0) : QRasterWindow(parent), m_color(color) {} + void paintEvent(QPaintEvent *) Q_DECL_OVERRIDE + { + QPainter p(this); + p.fillRect(QRect(QPoint(0, 0), size()), m_color); + } + +private: + const QColor m_color; +}; + +static bool isNativeWindowVisible(const QWindow *window) +{ +#if defined(Q_OS_WIN) && !defined(Q_OS_WINCE) && !defined(Q_OS_WINRT) + return IsWindowVisible(reinterpret_cast(window->winId())); +#else + Q_UNIMPLEMENTED(); + return window->isVisible(); +#endif +} + +void tst_QWindow::windowsTransientChildren() +{ + if (QGuiApplication::platformName().compare(QStringLiteral("windows"), Qt::CaseInsensitive)) + QSKIP("Windows only test"); + + ColoredWindow mainWindow(Qt::yellow); + mainWindow.setGeometry(QRect(m_availableTopLeft + QPoint(100, 100), m_testWindowSize)); + mainWindow.setTitle(QStringLiteral("Main")); + ColoredWindow child(Qt::blue, &mainWindow); + child.setGeometry(QRect(QPoint(0, 0), m_testWindowSize / 2)); + + ColoredWindow dialog(Qt::red); + dialog.setGeometry(QRect(m_availableTopLeft + QPoint(200, 200), m_testWindowSize)); + dialog.setTitle(QStringLiteral("Dialog")); + dialog.setTransientParent(&mainWindow); + + mainWindow.show(); + child.show(); + dialog.show(); + + QVERIFY(QTest::qWaitForWindowExposed(&dialog)); + mainWindow.setWindowState(Qt::WindowMinimized); + QVERIFY(!isNativeWindowVisible(&dialog)); + dialog.hide(); + mainWindow.setWindowState(Qt::WindowNoState); + // QTBUG-40696, transient children hidden by Qt should not be re-shown by Windows. + QVERIFY(!isNativeWindowVisible(&dialog)); + QVERIFY(isNativeWindowVisible(&child)); // Real children should be visible. +} + #include QTEST_MAIN(tst_QWindow) -- cgit v1.2.3