Merge lp://qastaging/~3v1n0/libindicator/shortcut-nicks-startup-notify into lp://qastaging/libindicator/13.10
Proposed by
Marco Trevisan (Treviño)
Status: | Merged |
---|---|
Approved by: | Mathieu Trudel-Lapierre |
Approved revision: | 486 |
Merged at revision: | 486 |
Proposed branch: | lp://qastaging/~3v1n0/libindicator/shortcut-nicks-startup-notify |
Merge into: | lp://qastaging/libindicator/13.10 |
Diff against target: |
57 lines (+19/-5) 1 file modified
libindicator/indicator-desktop-shortcuts.c (+19/-5) |
To merge this branch: | bzr merge lp://qastaging/~3v1n0/libindicator/shortcut-nicks-startup-notify |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mathieu Trudel-Lapierre | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email: mp+157785@code.qastaging.launchpad.net |
Commit message
IndicatorDestko
Otherwise GAppInfo won't pass the relevant platform-data to the Application launched.
Plus, fix the docs.
Description of the change
In lp:~3v1n0/libindicator/launch-shortucts-nicks-with-ctx I've forgot to add a StartupNotify prameter inside the internal key-file that we create to generate a DesktopAppInfo. Without it, not all the relevant platform-data (such as the activation timestamp) will be passed to clients.
And we need it.
To post a comment you must log in.
PASSED: Continuous integration, rev:486 jenkins. qa.ubuntu. com/job/ libindicator- ci/3/ jenkins. qa.ubuntu. com/job/ libindicator- raring- amd64-ci/ 3 jenkins. qa.ubuntu. com/job/ libindicator- raring- armhf-ci/ 3
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins: 8080/job/ libindicator- ci/3/rebuild
http://