Merge lp:~uriboni/unity-2d/clean-up-dash-dbus into lp:unity-2d/3.0
Proposed by
Ugo Riboni
Status: | Merged |
---|---|
Approved by: | Olivier Tilloy |
Approved revision: | no longer in the source branch. |
Merged at revision: | 429 |
Proposed branch: | lp:~uriboni/unity-2d/clean-up-dash-dbus |
Merge into: | lp:unity-2d/3.0 |
Diff against target: |
249 lines (+78/-36) 10 files modified
.bzrignore (+1/-0) launcher/UnityApplications/placeentry.cpp (+1/-1) launcher/app/gesturehandler.cpp (+1/-1) panel/applets/homebutton/homebuttonapplet.cpp (+1/-1) places/app/CMakeLists.txt (+5/-0) places/app/dash.xml (+47/-0) places/app/dashdeclarativeview.cpp (+17/-1) places/app/dashdeclarativeview.h (+2/-2) places/app/places.cpp (+2/-29) places/app/unity-2d-places.service.in (+1/-1) |
To merge this branch: | bzr merge lp:~uriboni/unity-2d/clean-up-dash-dbus |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Olivier Tilloy (community) | code functional | Approve | |
Review via email: mp+52037@code.launchpad.net |
Description of the change
[dash] Avoid exposing the entire QObject to DBUS, expose instead only what we really use
To post a comment you must log in.
You need to update the other components to activate the service on the new, normalized interface name:
$ grep -rn "\"com. canonical. Unity2d\ "" * UnityApplicatio ns/placeentry. cpp:146: static const char* DASH_DBUS_SERVICE = "com.canonical. Unity2d" ; app/gesturehand ler.cpp: 120: QDBusInterface dashInterface( "com.canonical. Unity2d" , "/Dash", "com.canonical. Unity2d. Dash"); homebutton/ homebuttonapple t.cpp:34: static const char* DBUS_SERVICE = "com.canonical. Unity2d" ;
launcher/
launcher/
panel/applets/