Merge lp:~mterry/indicator-sound/grab-greeter-name into lp:indicator-sound/14.04

[Michael Terry]

Use correct dbus name for Unity Greeter.

Use correct name for Unity Greeter.

unity-greeter (on desktop) happens to claim com.canonical.Unity because it pretends to be Unity in order to implement the shutdown dialog.

But really, it provides its own name com.canonical.UnityGreeter that should be used. For example, unity8-greeter doesn't currently claim com.canonical.Unity so the media player bits won't work on it until this change.

[Michael Terry (mterry)]

Oh and I fixed a method prototype that should allow a null invalidated property list.

[Michael Terry (mterry)]

In 14.10 we should totally fix unity-greeter and we can clean this code a bit.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:437
http://jenkins.qa.ubuntu.com/job/indicator-sound-ci/128/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/indicator-sound-trusty-amd64-ci/68/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-sound-trusty-armhf-ci/67
        deb: http://jenkins.qa.ubuntu.com/job/indicator-sound-trusty-armhf-ci/67/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/indicator-sound-ci/128/rebuild

review: Needs Fixing (continuous-integration)

[lp:~mterry/indicator-sound/grab-greeter-name]

438. By Michael Terry on 2014-04-01

Go back to useing correct name for unity greeter (especially in media player)

439. By Michael Terry on 2014-04-01

Drop minor null-allowed fix; let's not mix changes

[Ted Gould (ted)]

review: Approve

=== modified file 'src/media-player-list-greeter.vala'
--- src/media-player-list-greeter.vala	2014-03-03 19:28:24 +0000
+++ src/media-player-list-greeter.vala	2014-04-01 21:13:33 +0000
@@ -31,7 +31,7 @@
 	public MediaPlayerListGreeter () {
 		Bus.get_proxy.begin<UnityGreeterList> (
 			BusType.SESSION,
-			"com.canonical.Unity",
+			"com.canonical.UnityGreeter",
 			"/list",
 			DBusProxyFlags.NONE,
 			null,