Merge lp:~azzar1/unity/set-startup-id into lp:unity
Proposed by
Andrea Azzarone
Status: | Merged |
---|---|
Approved by: | Marco Trevisan (Treviño) |
Approved revision: | no longer in the source branch. |
Merged at revision: | 3056 |
Proposed branch: | lp:~azzar1/unity/set-startup-id |
Merge into: | lp:unity |
Diff against target: |
123 lines (+39/-5) 3 files modified
launcher/ApplicationLauncherIcon.cpp (+12/-5) launcher/ApplicationLauncherIcon.h (+1/-0) tests/autopilot/unity/tests/launcher/test_icon_behavior.py (+26/-0) |
To merge this branch: | bzr merge lp:~azzar1/unity/set-startup-id |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Needs Fixing | |
Thomi Richards (community) | quality | Approve | |
Marco Trevisan (Treviño) | Approve | ||
Review via email: mp+143624@code.launchpad.net |
Commit message
Set startupId when launching application from the launcher.
Description of the change
== Problem ==
Removal of 21_correct_
== Fix ==
Set startupId when launching application from the launcher.
Please not that something similar needs to be done for the apps lens too.
== Test ==
AP test added.
To post a comment you must log in.
92 + if self.app_ is_running( "Calculator" ): "Calculator is already running.")
93 + self.skipTest(
Why not closing it if, instead of skipping the test?
Anyway looks good.