Don't clear focus if setParent doesn't change the parent

QWidget::setParent might be called to change the window flags, without
changing the parent. For those cases, we don't have to clear the focus.

Decouple the newParent state from the wasCreated flag. In most places
where newParent was tested, wasCreated was either tested previously and
can't be false anyway, or the code executed is irrelevant for widgets
that are not yet created (there can't be a paint manager). In the
remaining case, test wasCreated explicitly to maintain existing logic.

Add test for the cases where the previous code broke the focus, both
for QWidget and QDialog.

Fixes: QTBUG-93005
Pick-to: 6.2
Change-Id: I39dc179c2d348054de3927aa8b69eecef4935511
Reviewed-by: Richard Moe Gustavsen <richard.gustavsen@qt.io>
Reviewed-by: Doris Verria <doris.verria@qt.io>

1 files changed, 72 insertions, 0 deletions

diff --git a/tests/auto/widgets/kernel/qwidget/tst_qwidget.cpp b/tests/auto/widgets/kernel/qwidget/tst_qwidget.cpp
index 2d77e4af93..8acba6748c 100644
--- a/tests/auto/widgets/kernel/qwidget/tst_qwidget.cpp
+++ b/tests/auto/widgets/kernel/qwidget/tst_qwidget.cpp
@@ -431,6 +431,9 @@ private slots:
     void deleteWindowInCloseEvent();
     void quitOnClose();
 
+    void setParentChangesFocus_data();
+    void setParentChangesFocus();
+
 private:
     bool ensureScreenSize(int width, int height);
 
@@ -12188,5 +12191,74 @@ void tst_QWidget::quitOnClose()
     QCOMPARE(quitSpy.count(), 2);
 }
 
+void tst_QWidget::setParentChangesFocus_data()
+{
+    QTest::addColumn<Qt::WindowType>("initialType");
+    QTest::addColumn<bool>("initialParent");
+    QTest::addColumn<Qt::WindowType>("targetType");
+    QTest::addColumn<bool>("targetParent");
+    QTest::addColumn<bool>("reparentBeforeShow");
+    QTest::addColumn<QString>("focusWidget");
+
+    for (const bool before : {true, false}) {
+        const char *tag = before ? "before" : "after";
+        QTest::addRow("give dialog parent, %s", tag)
+            << Qt::Dialog << false << Qt::Dialog << true << before << "lineEdit";
+        QTest::addRow("make dialog parentless, %s", tag)
+            << Qt::Dialog << true << Qt::Dialog << false << before << "lineEdit";
+        QTest::addRow("dialog to sheet, %s", tag)
+            << Qt::Dialog << true << Qt::Sheet << true << before << "lineEdit";
+        QTest::addRow("window to widget, %s", tag)
+            << Qt::Window << true << Qt::Widget << true << before << "windowEdit";
+        QTest::addRow("widget to window, %s", tag)
+            << Qt::Widget << true << Qt::Window << true << before << "lineEdit";
+    }
+}
+
+void tst_QWidget::setParentChangesFocus()
+{
+    QFETCH(Qt::WindowType, initialType);
+    QFETCH(bool, initialParent);
+    QFETCH(Qt::WindowType, targetType);
+    QFETCH(bool, targetParent);
+    QFETCH(bool, reparentBeforeShow);
+    QFETCH(QString, focusWidget);
+
+    QWidget window;
+    window.setObjectName("window");
+    QLineEdit *windowEdit = new QLineEdit(&window);
+    windowEdit->setObjectName("windowEdit");
+    windowEdit->setFocus();
+
+    std::unique_ptr<QWidget> secondary(new QWidget(initialParent ? &window : nullptr, initialType));
+    secondary->setObjectName("secondary");
+    QLineEdit *lineEdit = new QLineEdit(secondary.get());
+    lineEdit->setObjectName("lineEdit");
+    QPushButton *pushButton = new QPushButton(secondary.get());
+    pushButton->setObjectName("pushButton");
+    lineEdit->setFocus();
+
+    window.show();
+    QVERIFY(QTest::qWaitForWindowActive(&window));
+
+    if (reparentBeforeShow) {
+        secondary->setParent(targetParent ? &window : nullptr, targetType);
+        // making a widget into a window doesn't set a focusWidget until shown
+        if (secondary->focusWidget())
+            QCOMPARE(secondary->focusWidget()->objectName(), focusWidget);
+    }
+    secondary->show();
+    QApplication::setActiveWindow(secondary.get());
+    QVERIFY(QTest::qWaitForWindowActive(secondary.get()));
+
+    if (!reparentBeforeShow) {
+        secondary->setParent(targetParent ? &window : nullptr, targetType);
+        secondary->show(); // reparenting hides, so show again
+        QApplication::setActiveWindow(secondary.get());
+        QVERIFY(QTest::qWaitForWindowActive(secondary.get()));
+    }
+    QCOMPARE(QApplication::focusWidget()->objectName(), focusWidget);
+}
+
 QTEST_MAIN(tst_QWidget)
 #include "tst_qwidget.moc"