Merge lp:~dobey/unity8/processing-indterminate into lp:unity8

Proposed by dobey
Status: Superseded
Proposed branch: lp:~dobey/unity8/processing-indterminate
Merge into: lp:unity8
Diff against target: 629 lines (+145/-62)
20 files modified
CMakeLists.txt (+1/-1)
debian/control (+6/-4)
plugins/Ubuntu/DownloadDaemonListener/CMakeLists.txt (+3/-1)
plugins/Ubuntu/DownloadDaemonListener/DownloadTracker.cpp (+2/-1)
plugins/Ubuntu/DownloadDaemonListener/DownloadTracker.h (+2/-1)
plugins/Ubuntu/DownloadDaemonListener/interface/downloadtrackeradaptor.cpp (+2/-2)
plugins/Ubuntu/DownloadDaemonListener/interface/downloadtrackeradaptor.h (+71/-4)
plugins/Ubuntu/DownloadDaemonListener/interface/metatypes.h (+3/-6)
qml/Dash/Previews/PreviewProgress.qml (+5/-1)
qml/Stages/TabletStage.qml (+1/-1)
tests/mocks/Ubuntu/DownloadDaemonListener/MockDownloadTracker.cpp (+3/-1)
tests/mocks/Ubuntu/DownloadDaemonListener/MockDownloadTracker.h (+2/-1)
tests/mocks/Unity/Application/ApplicationInfo.cpp (+0/-8)
tests/mocks/Unity/Application/ApplicationInfo.h (+0/-4)
tests/mocks/Unity/Application/ApplicationManager.cpp (+0/-10)
tests/plugins/Unity/Launcher/launchermodeltest.cpp (+0/-2)
tests/qmltests/Dash/Previews/tst_PreviewProgress.qml (+14/-1)
tests/qmltests/Stages/tst_TabletStage.qml (+6/-3)
tests/qmltests/tst_OrientedShell.qml (+10/-6)
tests/qmltests/tst_Shell.qml (+14/-4)
To merge this branch: bzr merge lp:~dobey/unity8/processing-indterminate
Reviewer Review Type Date Requested Status
Albert Astals Cid (community) Approve
Unity8 CI Bot continuous-integration Needs Fixing
Ubuntu Unity PS integration team packaging Pending
Review via email: mp+300123@code.launchpad.net

This proposal has been superseded by a proposal from 2016-08-08.

Commit message

Set progress bar indeterminate when processing signal received.

Description of the change

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

No.

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

Yes.

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

Yes.

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

N/A

To post a comment you must log in.
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2543
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1746/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/2296/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2324
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2224
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2224
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2224
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2217/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2217/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2217/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2217/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2217/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2217/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2217/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2217/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2217/console

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1746/rebuild

review: Needs Fixing (continuous-integration)
2544. By dobey

Just include the system metatypes.h to always be up to date.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2544
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1753/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/2304/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2332
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2232
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2232
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2232
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2225/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2225/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2225/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2225/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2225/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2225/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2225/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2225/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2225/console

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1753/rebuild

review: Needs Fixing (continuous-integration)
2545. By dobey

Add libubuntu-download-manager-common-dev to build-depends for header.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2545
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1755/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/2306/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2334
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2233
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2233
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2233
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2226/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2226/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2226/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2226/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2226/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2226/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2226/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2226/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2226/console

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1755/rebuild

review: Needs Fixing (continuous-integration)
2546. By dobey

Need to link to udm-common.

2547. By dobey

The whitespace test is awful.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2546
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1759/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/2310/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2338
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2236
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2236
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2236
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2229/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2229/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2229/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2229/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2229/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2229/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2229/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2229/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2229/console

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1759/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2547
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1760/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2311
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1231
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1231
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1231
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2339
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2237
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2237
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2237
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2230
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2230/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2230
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2230/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2230
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2230/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2230
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2230/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2230
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2230/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2230
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2230/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2230
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2230/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2230
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2230/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2230
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2230/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1760/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

file:///home/tsdgeos_work/phablet/unity8/processing-indterminate/tests/qmltests/Dash/Previews/tst_PreviewProgress.qml:48:5: Type PreviewProgress unavailable
         PreviewProgress {
         ^
file:///home/tsdgeos_work/phablet/unity8/processing-indterminate/qml/Dash/Previews/PreviewProgress.qml:62:13: Cannot assign to non-existent property "onProcessing"
                 onProcessing: {

You need to update the mock DownloadTracker in tests/mocks/Ubuntu/DownloadDaemonListener/

review: Needs Fixing
2548. By dobey

Update the tests too.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
2549. By dobey

Processing doesn't call root triggered signal.

2550. By dobey

[ Daniel d'Andrada ]
Make use of MirSurface::inputBounds and drop UbuntuKeyboardInfo hack

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2550
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1790/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2344
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1252
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1252
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1252
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2372
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2258
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2258
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2258
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2251
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2251/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2251
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2251/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2251
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2251/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2251
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2251/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2251
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2251/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2251
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2251/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2251
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2251/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2251
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2251/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2251
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2251/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1790/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

previewProgress.progressBar doesn't exist, you need to use findChild as i suggested.

review: Needs Fixing
2551. By dobey

Use findChild instead.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2551
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1809/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2364
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1271
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1271
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1271
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2392
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2279
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2279
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2279
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2272
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2272/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2272
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2272/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2272
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2272/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2272
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2272/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2272
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2272/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2272
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2272/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2272
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2272/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2272
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2272/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2272
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2272/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1809/rebuild

review: Needs Fixing (continuous-integration)
2552. By dobey

Update copyright years in chagned files.
Do a second findChild in the test.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2552
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1824/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2381
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1282
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1282
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1282
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2409
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2296
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2296
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2296
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2289
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2289/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2289
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2289/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2289
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2289/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2289
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2289/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2289
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2289/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2289
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2289/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2289
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2289/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2289
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2289/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2289
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2289/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1824/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

the tryCompare call is wrong, you need 3 params there

tryCompare(progressBar, "indeterminate", true);

review: Needs Fixing
2553. By dobey

tryCompare is apparently inconsistent with Compare.

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2553
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1836/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2394
    FAILURE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1294/console
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1294
    FAILURE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1294/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2422
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2309
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2309
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2309
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2302
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2302/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2302
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2302/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2302
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2302/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2302
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2302/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2302
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2302/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2302
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2302/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2302
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2302/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2302
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2302/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2302
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2302/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1836/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

Have you shown this to someone in Design? In my opinion (which doesn't count much since i'm not in Design :D) this looks worse than just sticking at 100% for a few seconds more.

review: Needs Information
2554. By dobey

[ Zoltán Balogh ]
* Bump the version to secure UITK compatibility
[ Albert Astals Cid ]
* Pass the correct number of arguments to mapToItem (LP: #1606835)
* Rebuild against Qt 5.6.
* Add https://code.launchpad.net/~aacid/unity8/floor_includes/+merge/300850

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2554
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1849/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2413
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1307
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1307
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1307
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2441
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2328
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2328
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2328
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2321
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2321/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2321
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2321/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2321
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2321/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2321
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2321/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2321
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2321/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2321
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2321/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2321
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2321/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2321
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2321/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2321
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2321/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1849/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
dobey (dobey) wrote :

> Have you shown this to someone in Design? In my opinion (which doesn't count
> much since i'm not in Design :D) this looks worse than just sticking at 100%
> for a few seconds more.

I asked Paty to comment on the bug. FWIW, switching to indeterminate is also what Android does in the Play Store when download is finished and install starts. If the current result of that on Ubuntu "looks worse" as you say, then this seems like a problem with how indeterminate state is rendered, and thus a bug with UITK, and not this branch.

Revision history for this message
Albert Astals Cid (aacid) wrote :

Ok, let's show it to the world and if they disagree we fix it later on.

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

 * Did CI run pass? If not, please explain why.
Yes

review: Approve
Revision history for this message
Michał Sawicz (saviq) wrote :

Can you please resubmit with lp:~dandrader/unity8/removeApplicationStage as a prerequisite, they conflict in debian/control.

While at it, if you could please undo the two hunks where the generated file gained:

../../../../../../ubuntu-download-manager/trunk/docs/dbus/com.canonical.applications.download.xml

Or make those paths absolute, as appropriate, I'd be grateful.

Revision history for this message
dobey (dobey) wrote :

> Can you please resubmit with lp:~dandrader/unity8/removeApplicationStage as a
> prerequisite, they conflict in debian/control.
>
> While at it, if you could please undo the two hunks where the generated file
> gained:
>
> ../../../../../../ubuntu-download-
> manager/trunk/docs/dbus/com.canonical.applications.download.xml
>
> Or make those paths absolute, as appropriate, I'd be grateful.

This is because the u-d-m packages do not include the dbus API definition xml files in the packages, and so the ../../../ mess is pointing to a checkout in my home directory. Full paths would not be helpful here, without the xml files being installed in the u-d-m packaging anywhere. I've filed bug #1611051 about this issue, so hopefully we can instead add a build rule to make this easier to maintain in the future, and it will have sensible paths in the generated code.

2555. By dobey

Merged dandrader's branch.

2556. By dobey

[ Albert Astals Cid ]
* Take into account carousel selectedItemScaleFactor when setting card
  fixedArtShapeSize (LP: #1599238)
* Remove unused LimitProxyModel
* Do not calculate implicitHeight for Cards in a CardGrid
* Small clazy fixes
[ Daniel d'Andrada ]
* Remove stage property from Application
[ Larry Price ]
* Allow libertine-scope to show empty search result hint unmodified.
  (LP: #1606693)
[ Lukáš Tinkl ]
* Open the quicklist (context) menu also using the Menu key (LP:
  #1608265)
* Fix incorrect SIM PIN dialog position when entering a wrong PIN (LP:
  #1596076)
* Apply the correct keymap also on prompt surfaces (LP: #1610124)
* Implement an optional system update feature during OOBE wizard (LP:
  #1580785)
* Implement a page for choosing HW keyboard layout in OOBE wizard
[ Michael Zanetti ]
* Clean up some build script legacy
* Add a crossbuilder_post file to make crossbuilder restart unity8
  after deploying
* fixes for the automatic switching between usage modes (LP: #1590944)
[ Omer Akram ]
* Add object names for setup wizard
[ Albert Astals Cid ]
* Fix DisabledScreenNotice::test_rotation
* Make the delegate of DashNavigationList an async Loader
* unity-scope-tool: Use parse instead of process
* VerticalJournal improvements regarding model insertions and item
  height changes (LP: #1599754)
* Add math.h includes for compilation in yakkety (LP: #1605502)
[ Andrea Cimitan ]
* Unfocus the search text field in the dash page header when requested
  (LP: #1590820)
* Use binding for filter popover contentWidth (LP: #1595116)
* Add few sourceSize for Image we forgot (LP: #1595113)
* Replace Flickable with an Item inside DashPageHeader for
  headerContainer (LP: #1599235)
[ Albert Astals Cid, Andrea Cimitan ]
* Unbox the artshapeLoader
[ Daniel d'Andrada ]
* Fix splashscreen orientation when app rotates own contents (LP:
  #1586050)
* tst_PreviewIconActions: Make test name match filename
* Cursor: make its size grid unit based (LP: #1604014)
* TouchEventSequenceWrapper: item might get deleted when touch
  sequence is committed (LP: #1607686)
[ Lukáš Tinkl ]
* Implement frontend support for running keyboard indicator
* Provide window/surface close shortcuts across all the stages (LP:
  #1578392, #1606528)
* Restart the location trust prompt service when exiting the wizard
  (LP: #1594430)
* Implement clearing the search and closing the panel on pressing
  Escape key in the dash header
* Fix log timestamps (LP: #1602196)
* Don't display the mode switch warning dialog on tablets (LP:
  #1600290)
[ Michael Terry ]
* Fix indicator profile bug that caused sound indicator to say Mute
  instead of Silent Mode. (LP: #1604205)
* Fix lockscreen appearing right after unlocking a locked session on
  the desktop. (LP: #1604374)
[ Michał Sawicz ]
* Add arm64 support, drop unnecessary B-D on web plugin

Unmerged revisions

2556. By dobey

[ Albert Astals Cid ]
* Take into account carousel selectedItemScaleFactor when setting card
  fixedArtShapeSize (LP: #1599238)
* Remove unused LimitProxyModel
* Do not calculate implicitHeight for Cards in a CardGrid
* Small clazy fixes
[ Daniel d'Andrada ]
* Remove stage property from Application
[ Larry Price ]
* Allow libertine-scope to show empty search result hint unmodified.
  (LP: #1606693)
[ Lukáš Tinkl ]
* Open the quicklist (context) menu also using the Menu key (LP:
  #1608265)
* Fix incorrect SIM PIN dialog position when entering a wrong PIN (LP:
  #1596076)
* Apply the correct keymap also on prompt surfaces (LP: #1610124)
* Implement an optional system update feature during OOBE wizard (LP:
  #1580785)
* Implement a page for choosing HW keyboard layout in OOBE wizard
[ Michael Zanetti ]
* Clean up some build script legacy
* Add a crossbuilder_post file to make crossbuilder restart unity8
  after deploying
* fixes for the automatic switching between usage modes (LP: #1590944)
[ Omer Akram ]
* Add object names for setup wizard
[ Albert Astals Cid ]
* Fix DisabledScreenNotice::test_rotation
* Make the delegate of DashNavigationList an async Loader
* unity-scope-tool: Use parse instead of process
* VerticalJournal improvements regarding model insertions and item
  height changes (LP: #1599754)
* Add math.h includes for compilation in yakkety (LP: #1605502)
[ Andrea Cimitan ]
* Unfocus the search text field in the dash page header when requested
  (LP: #1590820)
* Use binding for filter popover contentWidth (LP: #1595116)
* Add few sourceSize for Image we forgot (LP: #1595113)
* Replace Flickable with an Item inside DashPageHeader for
  headerContainer (LP: #1599235)
[ Albert Astals Cid, Andrea Cimitan ]
* Unbox the artshapeLoader
[ Daniel d'Andrada ]
* Fix splashscreen orientation when app rotates own contents (LP:
  #1586050)
* tst_PreviewIconActions: Make test name match filename
* Cursor: make its size grid unit based (LP: #1604014)
* TouchEventSequenceWrapper: item might get deleted when touch
  sequence is committed (LP: #1607686)
[ Lukáš Tinkl ]
* Implement frontend support for running keyboard indicator
* Provide window/surface close shortcuts across all the stages (LP:
  #1578392, #1606528)
* Restart the location trust prompt service when exiting the wizard
  (LP: #1594430)
* Implement clearing the search and closing the panel on pressing
  Escape key in the dash header
* Fix log timestamps (LP: #1602196)
* Don't display the mode switch warning dialog on tablets (LP:
  #1600290)
[ Michael Terry ]
* Fix indicator profile bug that caused sound indicator to say Mute
  instead of Silent Mode. (LP: #1604205)
* Fix lockscreen appearing right after unlocking a locked session on
  the desktop. (LP: #1604374)
[ Michał Sawicz ]
* Add arm64 support, drop unnecessary B-D on web plugin

2555. By dobey

Merged dandrader's branch.

2554. By dobey

[ Zoltán Balogh ]
* Bump the version to secure UITK compatibility
[ Albert Astals Cid ]
* Pass the correct number of arguments to mapToItem (LP: #1606835)
* Rebuild against Qt 5.6.
* Add https://code.launchpad.net/~aacid/unity8/floor_includes/+merge/300850

2553. By dobey

tryCompare is apparently inconsistent with Compare.

2552. By dobey

Update copyright years in chagned files.
Do a second findChild in the test.

2551. By dobey

Use findChild instead.

2550. By dobey

[ Daniel d'Andrada ]
Make use of MirSurface::inputBounds and drop UbuntuKeyboardInfo hack

2549. By dobey

Processing doesn't call root triggered signal.

2548. By dobey

Update the tests too.

2547. By dobey

The whitespace test is awful.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2016-07-14 07:09:00 +0000
+++ CMakeLists.txt 2016-08-08 16:52:11 +0000
@@ -57,7 +57,7 @@
57find_package(Qt5Concurrent 5.4 REQUIRED)57find_package(Qt5Concurrent 5.4 REQUIRED)
58find_package(Qt5Sql 5.4 REQUIRED)58find_package(Qt5Sql 5.4 REQUIRED)
5959
60pkg_check_modules(APPLICATION_API REQUIRED unity-shell-application=19)60pkg_check_modules(APPLICATION_API REQUIRED unity-shell-application=20)
61pkg_check_modules(GEONAMES REQUIRED geonames>=0.2)61pkg_check_modules(GEONAMES REQUIRED geonames>=0.2)
62pkg_check_modules(GIO REQUIRED gio-2.0>=2.32)62pkg_check_modules(GIO REQUIRED gio-2.0>=2.32)
63pkg_check_modules(GLIB REQUIRED glib-2.0>=2.32)63pkg_check_modules(GLIB REQUIRED glib-2.0>=2.32)
6464
=== modified file 'debian/control'
--- debian/control 2016-08-03 15:53:00 +0000
+++ debian/control 2016-08-08 16:52:11 +0000
@@ -11,9 +11,9 @@
11# to g++ so we don't try to run arm g++11# to g++ so we don't try to run arm g++
12# on an x86 CPU for example, when cross-compiling.12# on an x86 CPU for example, when cross-compiling.
13 g++:native,13 g++:native,
14 libandroid-properties-dev,
15 graphviz,14 graphviz,
16 gsettings-ubuntu-schemas (>= 0.0.2+14.10.20140815),15 gsettings-ubuntu-schemas (>= 0.0.2+14.10.20140815),
16 libandroid-properties-dev,
17 libconnectivity-qt1-dev (>= 0.7.1),17 libconnectivity-qt1-dev (>= 0.7.1),
18 libevdev-dev,18 libevdev-dev,
19 libgeonames-dev (>= 0.2),19 libgeonames-dev (>= 0.2),
@@ -31,10 +31,12 @@
31 libqt5svg5-dev,31 libqt5svg5-dev,
32 libqt5xmlpatterns5-dev,32 libqt5xmlpatterns5-dev,
33 libsystemsettings-dev,33 libsystemsettings-dev,
34 libubuntu-download-manager-common-dev,
34 libubuntugestures5-dev (>= 1.3.2030),35 libubuntugestures5-dev (>= 1.3.2030),
35 libubuntugestures5-private-dev (>= 1.3.2030),36 libubuntugestures5-private-dev (>= 1.3.2030),
36 libudev-dev,37 libudev-dev,
37 libunity-api-dev (>= 7.116),38 libudm-common-dev,
39 libunity-api-dev (>= 7.117),
38 libusermetricsoutput1-dev,40 libusermetricsoutput1-dev,
39# Need those X11 libs touch emulation from mouse events in manual QML tests on a X11 desktop41# Need those X11 libs touch emulation from mouse events in manual QML tests on a X11 desktop
40 libx11-dev[!arm64 !armhf],42 libx11-dev[!arm64 !armhf],
@@ -152,7 +154,7 @@
152 qtdeclarative5-ubuntu-settings-components (>= 0.7),154 qtdeclarative5-ubuntu-settings-components (>= 0.7),
153 qtdeclarative5-unity-notifications-plugin (>= 0.1.2) | unity-notifications-impl,155 qtdeclarative5-unity-notifications-plugin (>= 0.1.2) | unity-notifications-impl,
154 ubuntu-thumbnailer-impl-0,156 ubuntu-thumbnailer-impl-0,
155 unity-application-impl-19,157 unity-application-impl-20,
156 unity-notifications-impl-3,158 unity-notifications-impl-3,
157 unity-plugin-scopes | unity-scopes-impl,159 unity-plugin-scopes | unity-scopes-impl,
158 unity-scopes-impl-12,160 unity-scopes-impl-12,
@@ -198,7 +200,7 @@
198Depends: ${misc:Depends},200Depends: ${misc:Depends},
199 ${shlibs:Depends},201 ${shlibs:Depends},
200Provides: unity-application-impl,202Provides: unity-application-impl,
201 unity-application-impl-19,203 unity-application-impl-20,
202Replaces: unity8-autopilot (<< 8.02+15.04.20150422-0ubuntu1)204Replaces: unity8-autopilot (<< 8.02+15.04.20150422-0ubuntu1)
203Description: Fake environment for running Unity 8 shell205Description: Fake environment for running Unity 8 shell
204 Provides fake implementations of some QML modules used by Unity 8 shell206 Provides fake implementations of some QML modules used by Unity 8 shell
205207
=== modified file 'plugins/Ubuntu/DownloadDaemonListener/CMakeLists.txt'
--- plugins/Ubuntu/DownloadDaemonListener/CMakeLists.txt 2016-06-02 09:32:33 +0000
+++ plugins/Ubuntu/DownloadDaemonListener/CMakeLists.txt 2016-08-08 16:52:11 +0000
@@ -12,5 +12,7 @@
12add_library(DownloadDaemonListener MODULE ${DOWNLOADDAEMONLISTENER_SOURCES})12add_library(DownloadDaemonListener MODULE ${DOWNLOADDAEMONLISTENER_SOURCES})
1313
14qt5_use_modules(DownloadDaemonListener Qml Quick DBus Core)14qt5_use_modules(DownloadDaemonListener Qml Quick DBus Core)
1515pkg_check_modules(DDLADAPTER REQUIRED udm-common)
16add_definitions(${DDLADAPTER_CFLAGS} ${DDLADAPTER_CFLAGS_OTHER})
17target_link_libraries(DownloadDaemonListener ${DDLADAPTER_LIBRARIES})
16add_unity8_plugin(Ubuntu.DownloadDaemonListener 0.1 Ubuntu/DownloadDaemonListener TARGETS DownloadDaemonListener)18add_unity8_plugin(Ubuntu.DownloadDaemonListener 0.1 Ubuntu/DownloadDaemonListener TARGETS DownloadDaemonListener)
1719
=== modified file 'plugins/Ubuntu/DownloadDaemonListener/DownloadTracker.cpp'
--- plugins/Ubuntu/DownloadDaemonListener/DownloadTracker.cpp 2015-08-19 13:56:21 +0000
+++ plugins/Ubuntu/DownloadDaemonListener/DownloadTracker.cpp 2016-08-08 16:52:11 +0000
@@ -1,5 +1,5 @@
1/*1/*
2 * Copyright (C) 2013 - Canonical Ltd.2 * Copyright (C) 2013-2016 Canonical Ltd.
3 *3 *
4 * This program is free software: you can redistribute it and/or modify it4 * This program is free software: you can redistribute it and/or modify it
5 * under the terms of the GNU Lesser General Public License, as5 * under the terms of the GNU Lesser General Public License, as
@@ -78,6 +78,7 @@
78 connect(m_adaptor, static_cast<void (DownloadTrackerAdaptor::*)(qulonglong, qulonglong)>(&DownloadTrackerAdaptor::progress), this, &DownloadTracker::progress);78 connect(m_adaptor, static_cast<void (DownloadTrackerAdaptor::*)(qulonglong, qulonglong)>(&DownloadTrackerAdaptor::progress), this, &DownloadTracker::progress);
79 connect(m_adaptor, &DownloadTrackerAdaptor::resumed, this, &DownloadTracker::resumed);79 connect(m_adaptor, &DownloadTrackerAdaptor::resumed, this, &DownloadTracker::resumed);
80 connect(m_adaptor, &DownloadTrackerAdaptor::started, this, &DownloadTracker::started);80 connect(m_adaptor, &DownloadTrackerAdaptor::started, this, &DownloadTracker::started);
81 connect(m_adaptor, &DownloadTrackerAdaptor::processing, this, &DownloadTracker::processing);
81 }82 }
82 // FIXME find a better way of determining if the service is ready83 // FIXME find a better way of determining if the service is ready
83 Q_EMIT serviceReadyChanged(m_adaptor && m_adaptor->isValid());84 Q_EMIT serviceReadyChanged(m_adaptor && m_adaptor->isValid());
8485
=== modified file 'plugins/Ubuntu/DownloadDaemonListener/DownloadTracker.h'
--- plugins/Ubuntu/DownloadDaemonListener/DownloadTracker.h 2015-11-20 15:01:39 +0000
+++ plugins/Ubuntu/DownloadDaemonListener/DownloadTracker.h 2016-08-08 16:52:11 +0000
@@ -1,5 +1,5 @@
1/*1/*
2 * Copyright (C) 2013 - Canonical Ltd.2 * Copyright (C) 2013-2016 Canonical Ltd.
3 *3 *
4 * This program is free software: you can redistribute it and/or modify it4 * This program is free software: you can redistribute it and/or modify it
5 * under the terms of the GNU Lesser General Public License, as5 * under the terms of the GNU Lesser General Public License, as
@@ -54,6 +54,7 @@
54 void error(const QString &error);54 void error(const QString &error);
55 void finished(const QString &path);55 void finished(const QString &path);
56 void paused(bool success);56 void paused(bool success);
57 void processing(const QString &path);
57 void progress(qulonglong received, qulonglong total);58 void progress(qulonglong received, qulonglong total);
58 void resumed(bool success);59 void resumed(bool success);
59 void started(bool success);60 void started(bool success);
6061
=== modified file 'plugins/Ubuntu/DownloadDaemonListener/interface/downloadtrackeradaptor.cpp'
--- plugins/Ubuntu/DownloadDaemonListener/interface/downloadtrackeradaptor.cpp 2013-07-24 14:18:23 +0000
+++ plugins/Ubuntu/DownloadDaemonListener/interface/downloadtrackeradaptor.cpp 2016-08-08 16:52:11 +0000
@@ -1,8 +1,8 @@
1/*1/*
2 * This file was generated by qdbusxml2cpp version 0.82 * This file was generated by qdbusxml2cpp version 0.8
3 * Command line was: qdbusxml2cpp -v -c DownloadTrackerAdaptor -p downloadtrackeradaptor.h:downloadtrackeradaptor.cpp -i metatypes.h com.canonical.applications.download.xml3 * Command line was: qdbusxml2cpp -v -c DownloadTrackerAdaptor -p downloadtrackeradaptor.h:downloadtrackeradaptor.cpp -i metatypes.h ../../../../../../ubuntu-download-manager/trunk/docs/dbus/com.canonical.applications.download.xml
4 *4 *
5 * qdbusxml2cpp is Copyright (C) 2013 Digia Plc and/or its subsidiary(-ies).5 * qdbusxml2cpp is Copyright (C) 2015 The Qt Company Ltd.
6 *6 *
7 * This is an auto-generated file.7 * This is an auto-generated file.
8 * This file may have been hand-edited. Look for HAND-EDIT comments8 * This file may have been hand-edited. Look for HAND-EDIT comments
99
=== modified file 'plugins/Ubuntu/DownloadDaemonListener/interface/downloadtrackeradaptor.h'
--- plugins/Ubuntu/DownloadDaemonListener/interface/downloadtrackeradaptor.h 2015-09-14 09:11:08 +0000
+++ plugins/Ubuntu/DownloadDaemonListener/interface/downloadtrackeradaptor.h 2016-08-08 16:52:11 +0000
@@ -1,15 +1,15 @@
1/*1/*
2 * This file was generated by qdbusxml2cpp version 0.82 * This file was generated by qdbusxml2cpp version 0.8
3 * Command line was: qdbusxml2cpp -v -c DownloadTrackerAdaptor -p downloadtrackeradaptor.h:downloadtrackeradaptor.cpp -i metatypes.h com.canonical.applications.download.xml3 * Command line was: qdbusxml2cpp -v -c DownloadTrackerAdaptor -p downloadtrackeradaptor.h:downloadtrackeradaptor.cpp -i metatypes.h ../../../../../../ubuntu-download-manager/trunk/docs/dbus/com.canonical.applications.download.xml
4 *4 *
5 * qdbusxml2cpp is Copyright (C) 2013 Digia Plc and/or its subsidiary(-ies).5 * qdbusxml2cpp is Copyright (C) 2015 The Qt Company Ltd.
6 *6 *
7 * This is an auto-generated file.7 * This is an auto-generated file.
8 * Do not edit! All changes made to it will be lost.8 * Do not edit! All changes made to it will be lost.
9 */9 */
1010
11#ifndef DOWNLOADTRACKERADAPTOR_H_137443437111#ifndef DOWNLOADTRACKERADAPTOR_H
12#define DOWNLOADTRACKERADAPTOR_H_137443437112#define DOWNLOADTRACKERADAPTOR_H
1313
14#include <QtCore/QObject>14#include <QtCore/QObject>
15#include <QtCore/QByteArray>15#include <QtCore/QByteArray>
@@ -36,6 +36,22 @@
3636
37 ~DownloadTrackerAdaptor();37 ~DownloadTrackerAdaptor();
3838
39 Q_PROPERTY(QString ClickPackage READ clickPackage)
40 inline QString clickPackage() const
41 { return qvariant_cast< QString >(property("ClickPackage")); }
42
43 Q_PROPERTY(QString DestinationApp READ destinationApp)
44 inline QString destinationApp() const
45 { return qvariant_cast< QString >(property("DestinationApp")); }
46
47 Q_PROPERTY(bool ShowInIndicator READ showInIndicator)
48 inline bool showInIndicator() const
49 { return qvariant_cast< bool >(property("ShowInIndicator")); }
50
51 Q_PROPERTY(QString Title READ title)
52 inline QString title() const
53 { return qvariant_cast< QString >(property("Title")); }
54
39public Q_SLOTS: // METHODS55public Q_SLOTS: // METHODS
40 inline QDBusPendingReply<> allowGSMDownload(bool allowed)56 inline QDBusPendingReply<> allowGSMDownload(bool allowed)
41 {57 {
@@ -50,6 +66,24 @@
50 return asyncCallWithArgumentList(QStringLiteral("cancel"), argumentList);66 return asyncCallWithArgumentList(QStringLiteral("cancel"), argumentList);
51 }67 }
5268
69 inline QDBusPendingReply<> collected()
70 {
71 QList<QVariant> argumentList;
72 return asyncCallWithArgumentList(QStringLiteral("collected"), argumentList);
73 }
74
75 inline QDBusPendingReply<QString> filePath()
76 {
77 QList<QVariant> argumentList;
78 return asyncCallWithArgumentList(QStringLiteral("filePath"), argumentList);
79 }
80
81 inline QDBusPendingReply<StringMap> headers()
82 {
83 QList<QVariant> argumentList;
84 return asyncCallWithArgumentList(QStringLiteral("headers"), argumentList);
85 }
86
53 inline QDBusPendingReply<bool> isGSMDownloadAllowed()87 inline QDBusPendingReply<bool> isGSMDownloadAllowed()
54 {88 {
55 QList<QVariant> argumentList;89 QList<QVariant> argumentList;
@@ -80,6 +114,27 @@
80 return asyncCallWithArgumentList(QStringLiteral("resume"), argumentList);114 return asyncCallWithArgumentList(QStringLiteral("resume"), argumentList);
81 }115 }
82116
117 inline QDBusPendingReply<> setDestinationDir(const QString &path)
118 {
119 QList<QVariant> argumentList;
120 argumentList << QVariant::fromValue(path);
121 return asyncCallWithArgumentList(QStringLiteral("setDestinationDir"), argumentList);
122 }
123
124 inline QDBusPendingReply<> setHeaders(StringMap headers)
125 {
126 QList<QVariant> argumentList;
127 argumentList << QVariant::fromValue(headers);
128 return asyncCallWithArgumentList(QStringLiteral("setHeaders"), argumentList);
129 }
130
131 inline QDBusPendingReply<> setMetadata(const QVariantMap &data)
132 {
133 QList<QVariant> argumentList;
134 argumentList << QVariant::fromValue(data);
135 return asyncCallWithArgumentList(QStringLiteral("setMetadata"), argumentList);
136 }
137
83 inline QDBusPendingReply<> setThrottle(qulonglong speed)138 inline QDBusPendingReply<> setThrottle(qulonglong speed)
84 {139 {
85 QList<QVariant> argumentList;140 QList<QVariant> argumentList;
@@ -93,6 +148,12 @@
93 return asyncCallWithArgumentList(QStringLiteral("start"), argumentList);148 return asyncCallWithArgumentList(QStringLiteral("start"), argumentList);
94 }149 }
95150
151 inline QDBusPendingReply<int> state()
152 {
153 QList<QVariant> argumentList;
154 return asyncCallWithArgumentList(QStringLiteral("state"), argumentList);
155 }
156
96 inline QDBusPendingReply<qulonglong> throttle()157 inline QDBusPendingReply<qulonglong> throttle()
97 {158 {
98 QList<QVariant> argumentList;159 QList<QVariant> argumentList;
@@ -106,10 +167,16 @@
106 }167 }
107168
108Q_SIGNALS: // SIGNALS169Q_SIGNALS: // SIGNALS
170 void authError(AuthErrorStruct error);
109 void canceled(bool success);171 void canceled(bool success);
110 void error(const QString &error);172 void error(const QString &error);
111 void finished(const QString &path);173 void finished(const QString &path);
174 void hashError(HashErrorStruct error);
175 void httpError(HttpErrorStruct error);
176 void networkError(NetworkErrorStruct error);
112 void paused(bool success);177 void paused(bool success);
178 void processError(ProcessErrorStruct error);
179 void processing(const QString &path);
113 void progress(qulonglong received, qulonglong total);180 void progress(qulonglong received, qulonglong total);
114 void resumed(bool success);181 void resumed(bool success);
115 void started(bool success);182 void started(bool success);
116183
=== modified file 'plugins/Ubuntu/DownloadDaemonListener/interface/metatypes.h'
--- plugins/Ubuntu/DownloadDaemonListener/interface/metatypes.h 2013-07-24 14:18:23 +0000
+++ plugins/Ubuntu/DownloadDaemonListener/interface/metatypes.h 2016-08-08 16:52:11 +0000
@@ -1,5 +1,5 @@
1/*1/*
2 * Copyright 2013 2013 Canonical Ltd.2 * Copyright 2013-2016 Canonical Ltd.
3 *3 *
4 * This library is free software; you can redistribute it and/or4 * This library is free software; you can redistribute it and/or
5 * modify it under the terms of version 3 of the GNU Lesser General Public5 * modify it under the terms of version 3 of the GNU Lesser General Public
@@ -19,10 +19,7 @@
19#ifndef DOWNLOADER_LIB_APP_METATYPES_H19#ifndef DOWNLOADER_LIB_APP_METATYPES_H
20#define DOWNLOADER_LIB_APP_METATYPES_H20#define DOWNLOADER_LIB_APP_METATYPES_H
2121
22#include <QMap>22// Just include the system file so we're always up to date.
2323#include <ubuntu/download_manager/metatypes.h>
24typedef QMap<QString, QString> StringMap;
25
26Q_DECLARE_METATYPE(StringMap)
2724
28#endif // METATYPES_H25#endif // METATYPES_H
2926
=== modified file 'qml/Dash/Previews/PreviewProgress.qml'
--- qml/Dash/Previews/PreviewProgress.qml 2015-07-15 15:07:19 +0000
+++ qml/Dash/Previews/PreviewProgress.qml 2016-08-08 16:52:11 +0000
@@ -1,5 +1,5 @@
1/*1/*
2 * Copyright (C) 2014 Canonical, Ltd.2 * Copyright (C) 2014-2016 Canonical, Ltd.
3 *3 *
4 * This program is free software; you can redistribute it and/or modify4 * This program is free software; you can redistribute it and/or modify
5 * it under the terms of the GNU General Public License as published by5 * it under the terms of the GNU General Public License as published by
@@ -59,6 +59,10 @@
59 }59 }
60 }60 }
6161
62 onProcessing: {
63 progressBar.indeterminate = true;
64 }
65
62 onFinished: {66 onFinished: {
63 root.triggered(widgetId, "finished", widgetData)67 root.triggered(widgetId, "finished", widgetData)
64 }68 }
6569
=== modified file 'qml/Stages/TabletStage.qml'
--- qml/Stages/TabletStage.qml 2016-07-12 09:43:50 +0000
+++ qml/Stages/TabletStage.qml 2016-08-08 16:52:11 +0000
@@ -542,7 +542,7 @@
542 if (!app) {542 if (!app) {
543 return index;543 return index;
544 }544 }
545 var stage = spreadRepeater.itemAt(index) ? spreadRepeater.itemAt(index).stage : app.stage;545 var stage = spreadRepeater.itemAt(index) ? spreadRepeater.itemAt(index).stage : ApplicationInfoInterface.MainStage;
546546
547 // don't shuffle indexes greater than "actives or next"547 // don't shuffle indexes greater than "actives or next"
548 if (index > 2) return index;548 if (index > 2) return index;
549549
=== modified file 'tests/mocks/Ubuntu/DownloadDaemonListener/MockDownloadTracker.cpp'
--- tests/mocks/Ubuntu/DownloadDaemonListener/MockDownloadTracker.cpp 2013-09-25 19:58:58 +0000
+++ tests/mocks/Ubuntu/DownloadDaemonListener/MockDownloadTracker.cpp 2016-08-08 16:52:11 +0000
@@ -1,5 +1,5 @@
1/*1/*
2 * Copyright (C) 2013 - Canonical Ltd.2 * Copyright (C) 2013-2016 Canonical Ltd.
3 *3 *
4 * This program is free software: you can redistribute it and/or modify it4 * This program is free software: you can redistribute it and/or modify it
5 * under the terms of the GNU Lesser General Public License, as5 * under the terms of the GNU Lesser General Public License, as
@@ -67,6 +67,8 @@
67 Q_EMIT finished("downloadComplete");67 Q_EMIT finished("downloadComplete");
68 }else if(m_dbusPath == "error") {68 }else if(m_dbusPath == "error") {
69 Q_EMIT error("DOWNLOAD ERROR");69 Q_EMIT error("DOWNLOAD ERROR");
70 }else if(m_dbusPath == "processing") {
71 Q_EMIT processing(m_dbusPath);
70 }72 }
71 }73 }
72}74}
7375
=== modified file 'tests/mocks/Ubuntu/DownloadDaemonListener/MockDownloadTracker.h'
--- tests/mocks/Ubuntu/DownloadDaemonListener/MockDownloadTracker.h 2013-08-13 08:50:50 +0000
+++ tests/mocks/Ubuntu/DownloadDaemonListener/MockDownloadTracker.h 2016-08-08 16:52:11 +0000
@@ -1,5 +1,5 @@
1/*1/*
2 * Copyright (C) 2013 - Canonical Ltd.2 * Copyright (C) 2013-2016 Canonical Ltd.
3 *3 *
4 * This program is free software: you can redistribute it and/or modify it4 * This program is free software: you can redistribute it and/or modify it
5 * under the terms of the GNU Lesser General Public License, as5 * under the terms of the GNU Lesser General Public License, as
@@ -51,6 +51,7 @@
51 void error(const QString &error);51 void error(const QString &error);
52 void finished(const QString &path);52 void finished(const QString &path);
53 void paused(bool success);53 void paused(bool success);
54 void processing(const QString &path);
54 void progress(qulonglong received, qulonglong total);55 void progress(qulonglong received, qulonglong total);
55 void resumed(bool success);56 void resumed(bool success);
56 void started(bool success);57 void started(bool success);
5758
=== modified file 'tests/mocks/Unity/Application/ApplicationInfo.cpp'
--- tests/mocks/Unity/Application/ApplicationInfo.cpp 2016-06-06 14:13:05 +0000
+++ tests/mocks/Unity/Application/ApplicationInfo.cpp 2016-08-08 16:52:11 +0000
@@ -175,14 +175,6 @@
175 }175 }
176}176}
177177
178void ApplicationInfo::setStage(Stage value)
179{
180 if (value != m_stage) {
181 m_stage = value;
182 Q_EMIT stageChanged(value);
183 }
184}
185
186void ApplicationInfo::setState(State value)178void ApplicationInfo::setState(State value)
187{179{
188 if (value != m_state) {180 if (value != m_state) {
189181
=== modified file 'tests/mocks/Unity/Application/ApplicationInfo.h'
--- tests/mocks/Unity/Application/ApplicationInfo.h 2016-06-03 11:11:34 +0000
+++ tests/mocks/Unity/Application/ApplicationInfo.h 2016-08-08 16:52:11 +0000
@@ -67,9 +67,6 @@
6767
68 QUrl icon() const override { return m_icon; }68 QUrl icon() const override { return m_icon; }
6969
70 Q_INVOKABLE void setStage(Stage value) override; // invokable only for mock
71 Stage stage() const override { return m_stage; }
72
73 Q_INVOKABLE void setState(State value);70 Q_INVOKABLE void setState(State value);
74 State state() const override { return m_state; }71 State state() const override { return m_state; }
7572
@@ -137,7 +134,6 @@
137 QString m_appId;134 QString m_appId;
138 QString m_name;135 QString m_name;
139 QUrl m_icon;136 QUrl m_icon;
140 Stage m_stage{MainStage};
141 State m_state{Stopped};137 State m_state{Stopped};
142 bool m_fullscreen{false};138 bool m_fullscreen{false};
143 Qt::ScreenOrientations m_supportedOrientations{Qt::PortraitOrientation |139 Qt::ScreenOrientations m_supportedOrientations{Qt::PortraitOrientation |
144140
=== modified file 'tests/mocks/Unity/Application/ApplicationManager.cpp'
--- tests/mocks/Unity/Application/ApplicationManager.cpp 2016-06-20 11:53:35 +0000
+++ tests/mocks/Unity/Application/ApplicationManager.cpp 2016-08-08 16:52:11 +0000
@@ -102,8 +102,6 @@
102 return app->comment();102 return app->comment();
103 case RoleIcon:103 case RoleIcon:
104 return app->icon();104 return app->icon();
105 case RoleStage:
106 return app->stage();
107 case RoleState:105 case RoleState:
108 return app->state();106 return app->state();
109 case RoleFocused:107 case RoleFocused:
@@ -166,11 +164,6 @@
166 if (!appIndex.isValid()) return;164 if (!appIndex.isValid()) return;
167 Q_EMIT dataChanged(appIndex, appIndex, QVector<int>() << ApplicationManager::RoleState);165 Q_EMIT dataChanged(appIndex, appIndex, QVector<int>() << ApplicationManager::RoleState);
168 });166 });
169 connect(application, &ApplicationInfo::stageChanged, this, [application, this]() {
170 QModelIndex appIndex = findIndex(application);
171 if (!appIndex.isValid()) return;
172 Q_EMIT dataChanged(appIndex, appIndex, QVector<int>() << ApplicationManager::RoleStage);
173 });
174167
175 connect(application, &ApplicationInfo::closed, this, [application, this]() {168 connect(application, &ApplicationInfo::closed, this, [application, this]() {
176 this->remove(application);169 this->remove(application);
@@ -344,7 +337,6 @@
344 application->setScreenshotId("gallery");337 application->setScreenshotId("gallery");
345 application->setIconId("gallery");338 application->setIconId("gallery");
346 application->setShellChrome(Mir::LowChrome);339 application->setShellChrome(Mir::LowChrome);
347 application->setStage(ApplicationInfo::MainStage);
348 m_availableApplications.append(application);340 m_availableApplications.append(application);
349341
350 application = new ApplicationInfo(this);342 application = new ApplicationInfo(this);
@@ -381,7 +373,6 @@
381 application->setName("GMail");373 application->setName("GMail");
382 application->setIconId("gmail");374 application->setIconId("gmail");
383 application->setScreenshotId("gmail-webapp.svg");375 application->setScreenshotId("gmail-webapp.svg");
384 application->setStage(ApplicationInfo::MainStage);
385 application->setSupportedOrientations(Qt::PortraitOrientation376 application->setSupportedOrientations(Qt::PortraitOrientation
386 | Qt::LandscapeOrientation377 | Qt::LandscapeOrientation
387 | Qt::InvertedPortraitOrientation378 | Qt::InvertedPortraitOrientation
@@ -393,7 +384,6 @@
393 application->setName("Music");384 application->setName("Music");
394 application->setIconId("soundcloud");385 application->setIconId("soundcloud");
395 application->setScreenshotId("music");386 application->setScreenshotId("music");
396 application->setStage(ApplicationInfo::MainStage);
397 application->setSupportedOrientations(Qt::PortraitOrientation387 application->setSupportedOrientations(Qt::PortraitOrientation
398 | Qt::LandscapeOrientation388 | Qt::LandscapeOrientation
399 | Qt::InvertedPortraitOrientation389 | Qt::InvertedPortraitOrientation
400390
=== modified file 'tests/plugins/Unity/Launcher/launchermodeltest.cpp'
--- tests/plugins/Unity/Launcher/launchermodeltest.cpp 2016-07-07 16:18:59 +0000
+++ tests/plugins/Unity/Launcher/launchermodeltest.cpp 2016-08-08 16:52:11 +0000
@@ -49,8 +49,6 @@
49 QString name() const override { return "mock"; }49 QString name() const override { return "mock"; }
50 QString comment() const override { return "this is a mock"; }50 QString comment() const override { return "this is a mock"; }
51 QUrl icon() const override { return QUrl(); }51 QUrl icon() const override { return QUrl(); }
52 ApplicationInfoInterface::Stage stage() const override { return ApplicationInfoInterface::MainStage; }
53 void setStage(ApplicationInfoInterface::Stage) override {}
54 ApplicationInfoInterface::State state() const override { return ApplicationInfoInterface::Running; }52 ApplicationInfoInterface::State state() const override { return ApplicationInfoInterface::Running; }
55 bool focused() const override { return m_focused; }53 bool focused() const override { return m_focused; }
56 QString splashTitle() const override { return QString(); }54 QString splashTitle() const override { return QString(); }
5755
=== modified file 'tests/qmltests/Dash/Previews/tst_PreviewProgress.qml'
--- tests/qmltests/Dash/Previews/tst_PreviewProgress.qml 2015-07-15 15:07:19 +0000
+++ tests/qmltests/Dash/Previews/tst_PreviewProgress.qml 2016-08-08 16:52:11 +0000
@@ -1,5 +1,5 @@
1/*1/*
2 * Copyright 2014 Canonical Ltd.2 * Copyright 2014-2016 Canonical Ltd.
3 *3 *
4 * This program is free software; you can redistribute it and/or modify4 * This program is free software; you can redistribute it and/or modify
5 * it under the terms of the GNU General Public License as published by5 * it under the terms of the GNU General Public License as published by
@@ -40,6 +40,11 @@
40 "source": { "dbus-name" : "somename", "dbus-object": "error" }40 "source": { "dbus-name" : "somename", "dbus-object": "error" }
41 }41 }
4242
43 property var progressjsonProcessing: {
44 "type": "progress",
45 "source": { "dbus-name" : "somename", "dbus-object": "processing" }
46 }
47
43 SignalSpy {48 SignalSpy {
44 id: spy49 id: spy
45 signalName: "triggered"50 signalName: "triggered"
@@ -82,6 +87,14 @@
82 compare(args[0], "previewProgress");87 compare(args[0], "previewProgress");
83 compare(args[1], "failed");88 compare(args[1], "failed");
84 compare(args[2], progressjsonError);89 compare(args[2], progressjsonError);
90
91 spy.clear();
92
93 var progressBar = findChild(previewProgress, "progressBar");
94 compare(progressBar.indeterminate, false);
95 previewProgress.widgetData = progressjsonProcessing;
96 progressBar = findChild(previewProgress, "progressBar");
97 tryCompare(progressBar, "indeterminate", true);
85 }98 }
86 }99 }
87}100}
88101
=== modified file 'tests/qmltests/Stages/tst_TabletStage.qml'
--- tests/qmltests/Stages/tst_TabletStage.qml 2016-06-22 13:42:30 +0000
+++ tests/qmltests/Stages/tst_TabletStage.qml 2016-08-08 16:52:11 +0000
@@ -290,8 +290,9 @@
290 var webbrowserSurfaceId = topSurfaceList.nextId;290 var webbrowserSurfaceId = topSurfaceList.nextId;
291 webbrowserCheckBox.checked = true;291 webbrowserCheckBox.checked = true;
292 waitUntilAppSurfaceShowsUp(webbrowserSurfaceId);292 waitUntilAppSurfaceShowsUp(webbrowserSurfaceId);
293 var webbrowserApp = ApplicationManager.findApplication(webbrowserCheckBox.appId);293 var webbrowserDelegate = findChild(tabletStage, "spreadDelegate_" + webbrowserSurfaceId);
294 compare(webbrowserApp.stage, ApplicationInfoInterface.MainStage);294 verify(webbrowserDelegate);
295 compare(webbrowserDelegate.stage, ApplicationInfoInterface.MainStage);
295 var webbrowserWindow = findAppWindowForSurfaceId(webbrowserSurfaceId);296 var webbrowserWindow = findAppWindowForSurfaceId(webbrowserSurfaceId);
296 verify(webbrowserWindow);297 verify(webbrowserWindow);
297298
@@ -397,8 +398,10 @@
397 var webbrowserSurfaceId = topSurfaceList.nextId;398 var webbrowserSurfaceId = topSurfaceList.nextId;
398 webbrowserCheckBox.checked = true;399 webbrowserCheckBox.checked = true;
399 waitUntilAppSurfaceShowsUp(webbrowserSurfaceId);400 waitUntilAppSurfaceShowsUp(webbrowserSurfaceId);
401 var webbrowserDelegate = findChild(tabletStage, "spreadDelegate_" + webbrowserSurfaceId);
402 verify(webbrowserDelegate);
403 compare(webbrowserDelegate.stage, ApplicationInfoInterface.MainStage);
400 var webbrowserApp = ApplicationManager.findApplication(webbrowserCheckBox.appId);404 var webbrowserApp = ApplicationManager.findApplication(webbrowserCheckBox.appId);
401 compare(webbrowserApp.stage, ApplicationInfoInterface.MainStage);
402405
403 var gallerySurfaceId = topSurfaceList.nextId;406 var gallerySurfaceId = topSurfaceList.nextId;
404 galleryCheckBox.checked = true;407 galleryCheckBox.checked = true;
405408
=== modified file 'tests/qmltests/tst_OrientedShell.qml'
--- tests/qmltests/tst_OrientedShell.qml 2016-07-29 20:25:09 +0000
+++ tests/qmltests/tst_OrientedShell.qml 2016-08-08 16:52:11 +0000
@@ -515,7 +515,6 @@
515 compare(primaryDelegate.focus, true);515 compare(primaryDelegate.focus, true);
516 compare(primaryApp.rotatesWindowContents, false);516 compare(primaryApp.rotatesWindowContents, false);
517 compare(primaryApp.supportedOrientations, Qt.PrimaryOrientation);517 compare(primaryApp.supportedOrientations, Qt.PrimaryOrientation);
518 compare(primaryApp.stage, ApplicationInfoInterface.MainStage);
519518
520 tryCompareFunction(function(){return primaryDelegate.surface != null;}, true);519 tryCompareFunction(function(){return primaryDelegate.surface != null;}, true);
521 verify(checkAppSurfaceOrientation(primaryAppWindow, primaryApp, root.primaryOrientationAngle));520 verify(checkAppSurfaceOrientation(primaryAppWindow, primaryApp, root.primaryOrientationAngle));
@@ -559,7 +558,8 @@
559 compare(ApplicationManager.focusedApplicationId, "primary-oriented-app");558 compare(ApplicationManager.focusedApplicationId, "primary-oriented-app");
560 compare(primaryApp.rotatesWindowContents, false);559 compare(primaryApp.rotatesWindowContents, false);
561 compare(primaryApp.supportedOrientations, Qt.PrimaryOrientation);560 compare(primaryApp.supportedOrientations, Qt.PrimaryOrientation);
562 compare(primaryApp.stage, ApplicationInfoInterface.MainStage);561 var primaryDelegate = findChild(shell, "spreadDelegate_" + primarySurfaceId);
562 compare(primaryDelegate.stage, ApplicationInfoInterface.MainStage);
563563
564 tryCompareFunction(function(){return primaryApp.surfaceList.count > 0;}, true);564 tryCompareFunction(function(){return primaryApp.surfaceList.count > 0;}, true);
565565
@@ -599,7 +599,6 @@
599 compare(gmailApp.rotatesWindowContents, false);599 compare(gmailApp.rotatesWindowContents, false);
600 compare(gmailApp.supportedOrientations, Qt.PortraitOrientation | Qt.LandscapeOrientation600 compare(gmailApp.supportedOrientations, Qt.PortraitOrientation | Qt.LandscapeOrientation
601 | Qt.InvertedPortraitOrientation | Qt.InvertedLandscapeOrientation);601 | Qt.InvertedPortraitOrientation | Qt.InvertedLandscapeOrientation);
602 compare(gmailApp.stage, ApplicationInfoInterface.MainStage);
603602
604 // wait until it's able to rotate603 // wait until it's able to rotate
605 tryCompare(shell, "orientationChangesEnabled", true);604 tryCompare(shell, "orientationChangesEnabled", true);
@@ -724,7 +723,6 @@
724 compare(gmailApp.rotatesWindowContents, false);723 compare(gmailApp.rotatesWindowContents, false);
725 compare(gmailApp.supportedOrientations, Qt.PortraitOrientation | Qt.LandscapeOrientation724 compare(gmailApp.supportedOrientations, Qt.PortraitOrientation | Qt.LandscapeOrientation
726 | Qt.InvertedPortraitOrientation | Qt.InvertedLandscapeOrientation);725 | Qt.InvertedPortraitOrientation | Qt.InvertedLandscapeOrientation);
727 compare(gmailApp.stage, ApplicationInfoInterface.MainStage);
728726
729 waitUntilAppWindowIsFullyLoaded(gmailSurfaceId);727 waitUntilAppWindowIsFullyLoaded(gmailSurfaceId);
730728
@@ -736,7 +734,10 @@
736 compare(musicApp.rotatesWindowContents, false);734 compare(musicApp.rotatesWindowContents, false);
737 compare(musicApp.supportedOrientations, Qt.PortraitOrientation | Qt.LandscapeOrientation735 compare(musicApp.supportedOrientations, Qt.PortraitOrientation | Qt.LandscapeOrientation
738 | Qt.InvertedPortraitOrientation | Qt.InvertedLandscapeOrientation);736 | Qt.InvertedPortraitOrientation | Qt.InvertedLandscapeOrientation);
739 compare(musicApp.stage, ApplicationInfoInterface.MainStage);737 if (data.deviceName === "manta" || data.deviceName === "flo") {
738 var musicDelegate = findChild(shell, "spreadDelegate_" + musicSurfaceId);
739 compare(musicDelegate.stage, ApplicationInfoInterface.MainStage);
740 }
740741
741 waitUntilAppWindowIsFullyLoaded(musicSurfaceId);742 waitUntilAppWindowIsFullyLoaded(musicSurfaceId);
742 tryCompare(shell, "orientationChangesEnabled", true);743 tryCompare(shell, "orientationChangesEnabled", true);
@@ -889,6 +890,7 @@
889 ];890 ];
890 }891 }
891 function test_appInSideStageDoesntRotateOnStartUp(data) {892 function test_appInSideStageDoesntRotateOnStartUp(data) {
893 WindowStateStorage.saveStage("twitter-webapp", ApplicationInfoInterface.SideStage)
892 loadShell(data.deviceName);894 loadShell(data.deviceName);
893895
894 var twitterDelegate = null;896 var twitterDelegate = null;
@@ -905,9 +907,11 @@
905 signalSpy.target = null;907 signalSpy.target = null;
906 signalSpy.signalName = "runningChanged";908 signalSpy.signalName = "runningChanged";
907909
910 var twitterSurfaceId = topLevelSurfaceList.nextId;
908 var twitterApp = ApplicationManager.startApplication("twitter-webapp");911 var twitterApp = ApplicationManager.startApplication("twitter-webapp");
909 verify(twitterApp);912 verify(twitterApp);
910 twitterApp.stage = ApplicationInfoInterface.SideStage;913 var twitterDelegate = findChild(shell, "spreadDelegate_" + twitterSurfaceId);
914 compare(twitterDelegate.stage, ApplicationInfoInterface.SideStage);
911915
912 // ensure the mock twitter-webapp is as we expect916 // ensure the mock twitter-webapp is as we expect
913 compare(twitterApp.rotatesWindowContents, false);917 compare(twitterApp.rotatesWindowContents, false);
914918
=== modified file 'tests/qmltests/tst_Shell.qml'
--- tests/qmltests/tst_Shell.qml 2016-08-03 15:53:36 +0000
+++ tests/qmltests/tst_Shell.qml 2016-08-08 16:52:11 +0000
@@ -2103,8 +2103,13 @@
2103 waitUntilAppWindowIsFullyLoaded(app2SurfaceId);2103 waitUntilAppWindowIsFullyLoaded(app2SurfaceId);
21042104
2105 // Sanity checking2105 // Sanity checking
2106 compare(app1.stage, ApplicationInfoInterface.MainStage);2106 if (data.usageScenario === "tablet") {
2107 compare(app2.stage, ApplicationInfoInterface.MainStage);2107 var app1Delegate = findChild(shell, "spreadDelegate_" + app1SurfaceId);
2108 compare(app1Delegate.stage, ApplicationInfoInterface.MainStage);
2109
2110 var app2Delegate = findChild(shell, "spreadDelegate_" + app2SurfaceId);
2111 compare(app2Delegate.stage, ApplicationInfoInterface.MainStage);
2112 }
2108 verify(!app1.isTouchApp);2113 verify(!app1.isTouchApp);
21092114
2110 var app1Surface = app1.surfaceList.get(0);2115 var app1Surface = app1.surfaceList.get(0);
@@ -2139,8 +2144,13 @@
2139 waitUntilAppWindowIsFullyLoaded(app2SurfaceId);2144 waitUntilAppWindowIsFullyLoaded(app2SurfaceId);
21402145
2141 // Sanity checking2146 // Sanity checking
2142 compare(app1.stage, ApplicationInfoInterface.MainStage);2147 if (data.usageScenario === "tablet") {
2143 compare(app2.stage, ApplicationInfoInterface.MainStage);2148 var app1Delegate = findChild(shell, "spreadDelegate_" + app1SurfaceId);
2149 compare(app1Delegate.stage, ApplicationInfoInterface.MainStage);
2150
2151 var app2Delegate = findChild(shell, "spreadDelegate_" + app2SurfaceId);
2152 compare(app2Delegate.stage, ApplicationInfoInterface.MainStage);
2153 }
21442154
2145 var app1Surface = app1.surfaceList.get(0);2155 var app1Surface = app1.surfaceList.get(0);
2146 verify(app1Surface);2156 verify(app1Surface);

Subscribers

People subscribed via source and target branches