Schedule layout changes to avoid refill() recursion.

Otherwise we add duplicate item delegates.

Task-number: QTBUG-40298
Change-Id: I8a2221d72f458c892720a71d87513808a5d725a6
Reviewed-by: Martin Jones <martin.jones@jollamobile.com>

1 files changed, 45 insertions, 0 deletions

diff --git a/tests/auto/quick/qquickpathview/data/changePathDuringRefill.qml b/tests/auto/quick/qquickpathview/data/changePathDuringRefill.qml
new file mode 100644
index 0000000000..f02ab35faf
--- /dev/null
+++ b/tests/auto/quick/qquickpathview/data/changePathDuringRefill.qml
@@ -0,0 +1,45 @@
+import QtQuick 2.3
+
+PathView {
+    id: view
+    objectName: "pathView"
+    width: 100
+    height: delegateHeight * pathItemCount
+    model: ["A", "B", "C"]
+    pathItemCount: 3
+    anchors.centerIn: parent
+
+    property int delegateHeight: 0
+
+    activeFocusOnTab: true
+    Keys.onDownPressed: view.incrementCurrentIndex()
+    Keys.onUpPressed: view.decrementCurrentIndex()
+    preferredHighlightBegin: 0.5
+    preferredHighlightEnd: 0.5
+
+    delegate: Rectangle {
+        objectName: "delegate" + modelData
+        width: view.width
+        height: textItem.height
+        border.color: "red"
+
+        onHeightChanged: {
+            if (index == 0)
+                view.delegateHeight = textItem.height
+        }
+
+        Text {
+            id: textItem
+            text: modelData
+        }
+    }
+
+    path: Path {
+        startX: view.width / 2
+        startY: 0
+        PathLine {
+            x: view.width / 2
+            y: view.pathItemCount * view.delegateHeight
+        }
+    }
+}