Merge lp:~mardy/unity8/lp1433442 into lp:unity8

[Alberto Mardegan]

Set currentIndex after the model change is completed

When setting the model, the list view will first emit the countChanged signal, then will change the currentIndex to 0 (see QtDeclarative git commit 22c6873 and https://bugreports.qt.io/browse/QTBUG-26604), and finally will emit the modelChanged signal.
If we set the currentIndex in the onCountChanged handler (as we were doing
before), the currentIndex would be reset to 0 shortly afterwards.

For some mysterious reason, the desired delegate was anyway shown until the bugfix for bug 1433442 was introduced; with that bugfix, the listview is more consistent and shows the delegate corresponding to the currentIndex.

Set currentIndex after the model change is completed

When setting the model, the list view will first emit the countChanged signal, then will change the currentIndex to 0 (see QtDeclarative git commit 22c6873 and https://bugreports.qt.io/browse/QTBUG-26604), and finally will emit the modelChanged signal.
If we set the currentIndex in the onCountChanged handler (as we were doing
before), the currentIndex would be reset to 0 shortly afterwards.

For some mysterious reason, the desired delegate was anyway shown until the bugfix for bug 1433442 was introduced; with that bugfix, the listview is more consistent and shows the delegate corresponding to the currentIndex.

 * Are there any related MPs required for this MP to build/function as expected?

No, although this change is mostly relevant when using libqt5quick5 from silo 29 (because without this change, that silo would cause a regression).

 * Did you perform an exploratory manual test run of your code change and any related functionality?

Yes.

 * Did you make sure that your branch does not contain spurious tags?

Yes.

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?

It isn't.

 * If you changed the UI, has there been a design review?

No visible UI changes.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1917
http://jenkins.qa.ubuntu.com/job/unity8-ci/6196/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-wily-touch/546/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-wily/226/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-wily-amd64-ci/435/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-wily-i386-ci/436/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-wily-armhf/546/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/6196/rebuild

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

This is actually regressing (without the silo 29).

Is silo 29 ready/testable already?

review: Needs Fixing

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1918
http://jenkins.qa.ubuntu.com/job/unity8-ci/6197/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-wily-touch/547/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-wily/227/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-wily-amd64-ci/436/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-wily-i386-ci/437/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-wily-armhf/547/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/6197/rebuild

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

 * Did you perform an exploratory manual test run of the code change and any related functionality?
Yes

 * Did CI run pass?
It's kaboomed

 * Did you make sure that the branch does not contain spurious tags?
Yes

review: Approve

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1919
http://jenkins.qa.ubuntu.com/job/unity8-ci/6203/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-wily-touch/553/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-wily/233/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-wily-amd64-ci/442/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-wily-i386-ci/443/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-wily-armhf/553/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/6203/rebuild

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

review: Approve

=== modified file 'qml/Dash/PreviewListView.qml'
--- qml/Dash/PreviewListView.qml	2015-08-17 11:46:35 +0000
+++ qml/Dash/PreviewListView.qml	2015-08-17 11:46:35 +0000
@@ -86,10 +86,10 @@
             }
         }
 
-        onCountChanged: {
+        onModelChanged: {
             if (count > 0 && initialIndex >= 0 && !usedInitialIndex) {
                 usedInitialIndex = true;
-                currentIndex = initialIndex;
+                previewListView.positionViewAtIndex(initialIndex, ListView.SnapPosition);
             }
         }