Instantiator: don't interfere with delegates that assign parents

[ChangeLog][QtQml][Instantiator] Instantiator now avoids re-assigning a
delegate object's parent to itself if it was already set; thus, you can
now declare a parent assignment.

Task-number: QTBUG-64546
Task-number: QTBUG-84730
Change-Id: I7d95fa76e71c363b4cb5b7a512c2e984488c8af4
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Reviewed-by: Richard Moe Gustavsen <richard.gustavsen@qt.io>

1 files changed, 15 insertions, 0 deletions

diff --git a/tests/auto/qml/qqmlinstantiator/tst_qqmlinstantiator.cpp b/tests/auto/qml/qqmlinstantiator/tst_qqmlinstantiator.cpp
index 4c6c84a9d2..a0ab2cda5b 100644
--- a/tests/auto/qml/qqmlinstantiator/tst_qqmlinstantiator.cpp
+++ b/tests/auto/qml/qqmlinstantiator/tst_qqmlinstantiator.cpp
@@ -56,6 +56,8 @@ private slots:
 
     void asynchronous_data();
     void asynchronous();
+
+    void handlerWithParent();
 };
 
 tst_qqmlinstantiator::tst_qqmlinstantiator()
@@ -276,6 +278,19 @@ void tst_qqmlinstantiator::asynchronous()
     }
 }
 
+void tst_qqmlinstantiator::handlerWithParent()
+{
+    QQmlEngine engine;
+    QQmlComponent component(&engine, testFileUrl("handlerWithParent.qml"));
+    QObject *rootObject = component.create();
+    QVERIFY(rootObject != nullptr);
+    const auto handlers = rootObject->findChildren<QObject *>("pointHandler");
+    QCOMPARE(handlers.count(), 2);
+    for (const auto *h : handlers) {
+        QCOMPARE(h->parent(), rootObject);
+    }
+}
+
 QTEST_MAIN(tst_qqmlinstantiator)
 
 #include "tst_qqmlinstantiator.moc"