Integrate property binding evaluation fix from qtbase

This is basically the same as QTBUG-105204, only with the QML engine
being involved.

Fixes: QTBUG-104982
Pick-to: 6.4 6.3 6.2
Change-Id: I5afaadedcd7af41198702a8f2331703b4f6ef2e7
Reviewed-by: Sami Shalayel <sami.shalayel@qt.io>
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>

src/qml/qml/qqmlpropertybinding.cpp

@@ -131,7 +131,8 @@ QUntypedPropertyBinding QQmlPropertyBinding::createFromBoundFunction(const QQmlP
 
 void QQmlPropertyBindingJS::expressionChanged()
 {
-    if (!asBinding()->propertyDataPtr)
+    auto binding = asBinding();
+    if (!binding->propertyDataPtr)
         return;
     const auto currentTag = m_error.tag();
     if (currentTag == InEvaluationLoop) {
@@ -151,8 +152,9 @@ void QQmlPropertyBindingJS::expressionChanged()
         return;
     }
     m_error.setTag(InEvaluationLoop);
-    asBinding()->evaluateRecursive();
-    asBinding()->notifyRecursive();
+    PendingBindingObserverList bindingObservers;
+    binding->evaluateRecursive(bindingObservers);
+    binding->notifyNonRecursive(bindingObservers);
     m_error.setTag(NoTag);
 }
 

tests/auto/qml/qqmlecmascript/data/qpropertyBindingNoQPropertyCapture.qml

@@ -0,0 +1,19 @@
+import QtQuick
+
+Item {
+    objectName: "redRectangle"
+    id: redRectangle
+
+    property bool b: false
+    function toggle() { b = !b }
+    width: b ? 600 : 500
+
+    Item {
+        id: blueRectangle
+        objectName: "blueRectangle"
+        // width: b ? (100 + redRectangle.width / 2) : 25
+        width: b ? redRectangle.width : 25
+    }
+
+    property int blueRectangleWidth: blueRectangle.width
+}

tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp

@@ -292,6 +292,7 @@ private slots:
     void bindingBoundFunctions();
     void qpropertyAndQtBinding();
     void qpropertyBindingReplacement();
+    void qpropertyBindingNoQPropertyCapture();
     void deleteRootObjectInCreation();
     void onDestruction();
     void onDestructionViaGC();
@@ -7737,6 +7738,28 @@ void tst_qqmlecmascript::qpropertyBindingReplacement()
     QCOMPARE(root->objectName(), u"overwritten"_s);
 }
 
+void tst_qqmlecmascript::qpropertyBindingNoQPropertyCapture()
+{
+
+    QQmlEngine engine;
+    QQmlComponent comp(&engine, testFileUrl("qpropertyBindingNoQPropertyCapture.qml"));
+    std::unique_ptr<QObject> root(comp.create());
+    QVERIFY2(root, qPrintable(comp.errorString()));
+    auto redRectangle = root.get();
+
+    QQmlProperty blueRectangleWidth(redRectangle, "blueRectangleWidth", &engine);
+
+    auto toggle = [&](){
+        QMetaObject::invokeMethod(root.get(), "toggle");
+    };
+
+    QCOMPARE(blueRectangleWidth.read().toInt(), 25);
+    toggle();
+    QCOMPARE(blueRectangleWidth.read().toInt(), 600);
+    toggle();
+    QCOMPARE(blueRectangleWidth.read().toInt(), 25);
+}
+
 void tst_qqmlecmascript::deleteRootObjectInCreation()
 {
     QQmlEngine engine;