Merge lp://staging/~stolowski/unity/hide-dash-dbus-iface into lp://staging/unity
Proposed by
Paweł Stołowski
Status: | Merged |
---|---|
Approved by: | Michal Hruby |
Approved revision: | no longer in the source branch. |
Merged at revision: | 2678 |
Proposed branch: | lp://staging/~stolowski/unity/hide-dash-dbus-iface |
Merge into: | lp://staging/unity |
Diff against target: |
197 lines (+98/-1) 4 files modified
dash/DashController.cpp (+70/-0) dash/DashController.h (+10/-0) tests/autopilot/unity/emulators/dash.py (+9/-1) tests/autopilot/unity/tests/test_dash.py (+9/-0) |
To merge this branch: | bzr merge lp://staging/~stolowski/unity/hide-dash-dbus-iface |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michal Hruby (community) | Approve | ||
Review via email: mp+123271@code.staging.launchpad.net |
Commit message
Implemented DBus interface for DashController and made HideDash() method available via DBus. Added autopilot test for it.
This is needed by music-preview-
Description of the change
Implemented DBus interface for DashController and made HideDash() method available via DBus. Added autopilot test for it.
This is needed by music-preview-
To post a comment you must log in.
9 +const std::string DBUS_NAME = "com.canonical. Unity.Dash" ; bus.get_ object( 'com.canonical. Unity.Dash' ,
148 + dash_object = session_
I don't think we need to request yet another name for the dash, "com.canonical. Unity" (which is already used elsewhere in the code) is good enough.