Merge lp:~aacid/indicator-sound/kill_determine_key into lp:indicator-sound/12.10
Proposed by
Albert Astals Cid
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Conor Curran | ||||
Approved revision: | 334 | ||||
Merged at revision: | 335 | ||||
Proposed branch: | lp:~aacid/indicator-sound/kill_determine_key | ||||
Merge into: | lp:indicator-sound/12.10 | ||||
Diff against target: |
131 lines (+18/-40) 1 file modified
src/music-player-bridge.vala (+18/-40) |
||||
To merge this branch: | bzr merge lp:~aacid/indicator-sound/kill_determine_key | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Conor Curran (community) | Approve | ||
jenkins (community) | continuous-integration | Approve | |
Review via email: mp+128650@code.launchpad.net |
Commit message
Remove the determine_key function
As documented it did not work very well and it is not really needed since
they only time we need to map from dbus interface to desktop name is
when the program goes away that means we had previously seen it appearing
thus we can have a map with the dbus interface -> destkop name mapping
Description of the change
Remove the determine_key function
As documented it did not work very well and it is not really needed since
they only time we need to map from dbus interface to desktop name is
when the program goes away that means we had previously seen it appearing
thus we can have a map with the dbus interface -> destkop name mapping
To post a comment you must log in.
PASSED: Continuous integration, rev:334 jenkins. qa.ubuntu. com/job/ indicator- sound-ci/ 2/ jenkins. qa.ubuntu. com/job/ indicator- sound-ci/ ./label= quantal/ 2/console
http://
Executed test runs:
SUCCESS: http://