Make sure positioners remove change listeners for invisible items.

Children that are invisble weren't having their change listeners
removed, which showed illegal accesses on destruction in valgrind.

Change-Id: Icae798e773168323781e9ab88b3dae6a5aea0952
Reviewed-by: Alan Alpert <alan.alpert@nokia.com>

1 files changed, 5 insertions, 0 deletions

diff --git a/src/quick/items/qquickpositioners.cpp b/src/quick/items/qquickpositioners.cpp
index 77d2a360b8..95ee9bfb58 100644
--- a/src/quick/items/qquickpositioners.cpp
+++ b/src/quick/items/qquickpositioners.cpp
@@ -111,6 +111,8 @@ QQuickBasePositioner::~QQuickBasePositioner()
     Q_D(QQuickBasePositioner);
     for (int i = 0; i < positionedItems.count(); ++i)
         d->unwatchChanges(positionedItems.at(i).item);
+    for (int i = 0; i < unpositionedItems.count(); ++i)
+        d->unwatchChanges(unpositionedItems.at(i).item);
     positionedItems.clear();
 }
 
@@ -187,6 +189,9 @@ void QQuickBasePositioner::itemChange(ItemChange change, const ItemChangeData &v
         if (idx >= 0) {
             d->unwatchChanges(child);
             positionedItems.remove(idx);
+        } else if ((idx = unpositionedItems.find(posItem)) >= 0) {
+            d->unwatchChanges(child);
+            unpositionedItems.remove(idx);
         }
         d->setPositioningDirty();
     }