Merge lp:~zeller-benjamin/unity8/scope-url into lp:unity8

[Benjamin Zeller]

Add support for scope:// url in the dash

 * Are there any related MPs required for this MP to build/function as expected? Please list.
lp:~ted/url-dispatcher/scope-uri needed to work, but can land separately

 * Did you perform an exploratory manual test run of your code change and any related functionality?
Y
 * Did you make sure that your branch does not contain spurious tags?
Y
 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A
 * If you changed the UI, has there been a design review?
N/A

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1185
http://jenkins.qa.ubuntu.com/job/unity8-ci/4060/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/3922/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/1061/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/1154/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/1154/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/5169/console

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

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1186
http://jenkins.qa.ubuntu.com/job/unity8-ci/4062/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/3946
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/1063
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/1156
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/1156
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/3795
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/5193
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/5193/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/11946

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

review: Needs Fixing (continuous-integration)

[Michał Sawicz (saviq)]

Hey, you can drop the .url-dispatcher file after all, scope:// will be handled in a custom way towards unity8-dash.

review: Needs Fixing

[Michał Sawicz (saviq)]

Apparently Ted still will parse the file, just the activation will be special.

review: Abstain

[Ted Gould (ted)]

Actually, I think having the url-dispatcher file is a good idea. Then the dash can configure which URLs it wants in case others are needed later. I'll just special case the unity8-dash name instead of which URL patterns are associated with it.

[Michał Sawicz (saviq)]

Huum, can't see how to test this atm, we'd need to mock UITK ;|

There should probably be a platform test for this instead.

 * Did you perform an exploratory manual test run of the code change and any related functionality?
Y
 * Did CI run pass? If not, please explain why.
Is running now, only failed test before is fixed now in trunk

review: Approve

[Michał Sawicz (saviq)]

review: Needs Fixing

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:1187
http://jenkins.qa.ubuntu.com/job/unity8-ci/4112/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/4070
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/1113
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/1206
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/1206
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/3906
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/5322
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/5322/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/12146

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

review: Approve (continuous-integration)

[lp:~zeller-benjamin/unity8/scope-url]

1188. By Benjamin Zeller on 2014-08-27

Move url-dispatcher file from common to unity package

[Michał Sawicz (saviq)]

review: Approve

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:1188
http://jenkins.qa.ubuntu.com/job/unity8-ci/4122/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/4095
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/1123
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/1216
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/1216
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/3932
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/5347
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/5347/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/12187

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

review: Approve (continuous-integration)

=== modified file 'data/CMakeLists.txt'
--- data/CMakeLists.txt	2014-07-23 06:54:10 +0000
+++ data/CMakeLists.txt	2014-08-27 08:03:23 +0000
@@ -8,3 +8,6 @@
     ${CMAKE_CURRENT_BINARY_DIR}/${DASH_APP}.desktop
     DESTINATION ${CMAKE_INSTALL_DATADIR}/applications
     )
+
+# install url dispatcher files
+install(FILES unity8-dash.url-dispatcher DESTINATION ${CMAKE_INSTALL_DATADIR}/url-dispatcher/urls)

=== modified file 'data/unity8-dash.conf'
--- data/unity8-dash.conf	2014-08-06 08:17:31 +0000
+++ data/unity8-dash.conf	2014-08-27 08:03:23 +0000
@@ -10,6 +10,8 @@
 
 respawn
 
+env APP_ID=unity8-dash
+
 pre-start script
     if [ -z "$UNITY_SCOPES_LIST" ]; then
         # FIXME: remove once we have this in dconf

=== added file 'data/unity8-dash.url-dispatcher'
--- data/unity8-dash.url-dispatcher	1970-01-01 00:00:00 +0000
+++ data/unity8-dash.url-dispatcher	2014-08-27 08:03:23 +0000
@@ -0,0 +1,6 @@
+[
+        {
+                "protocol": "scope"
+        }
+]
+

=== modified file 'debian/unity8.install'
--- debian/unity8.install	2014-07-23 16:25:19 +0000
+++ debian/unity8.install	2014-08-27 08:03:23 +0000
@@ -10,3 +10,4 @@
 usr/share/unity8/Panel
 usr/share/unity8/Shell.qml
 usr/share/unity8/Stages
+usr/share/url-dispatcher/urls/unity8-dash.url-dispatcher

=== modified file 'qml/Dash/DashContent.qml'
--- qml/Dash/DashContent.qml	2014-08-08 08:47:36 +0000
+++ qml/Dash/DashContent.qml	2014-08-27 08:03:23 +0000
@@ -52,6 +52,11 @@
         }
     }
 
+    Connections {
+        target: UriHandler
+        onOpened: dashContentList.currentItem.theScope.performQuery(uris[0])
+    }
+
     function setCurrentScopeAtIndex(index, animate, reset) {
         // if the scopes haven't loaded yet, then wait until they are.
         if (!scopes.loaded) {