Ensure view delegates are object parented to the contentItem

Caused a leak which resulted in persistent v8 handles not being
disposed.

Change-Id: I5c10ff619531976ab61d9ce1ff8327c4a40cfd5e
Reviewed-on: http://codereview.qt-project.org/6112
Reviewed-by: Qt Sanity Bot <qt_sanity_bot@ovi.com>
Reviewed-by: Chris Adams <christopher.adams@nokia.com>

diff --git a/src/declarative/items/qsgitemview.cpp b/src/declarative/items/qsgitemview.cpp
index 3224e32..b34ed82 100644 (file)
--- a/src/declarative/items/qsgitemview.cpp
+++ b/src/declarative/items/qsgitemview.cpp
@@ -1578,9 +1578,11 @@ FxViewItem *QSGItemViewPrivate::createItem(int modelIndex)
             if (model->completePending()) {
                 // complete
                 viewItem->item->setZ(1);
+                QDeclarative_setParent_noEvent(viewItem->item, q->contentItem());
                 viewItem->item->setParentItem(q->contentItem());
                 model->completeItem();
             } else {
+                QDeclarative_setParent_noEvent(viewItem->item, q->contentItem());
                 viewItem->item->setParentItem(q->contentItem());
             }
             // do other set up for the new item that should not happen