Merge lp://qastaging/~gerboland/qtmir/fix-unfocus-notification into lp://qastaging/qtmir

Proposed by Gerry Boland
Status: Merged
Approved by: Daniel d'Andrada
Approved revision: 224
Merged at revision: 229
Proposed branch: lp://qastaging/~gerboland/qtmir/fix-unfocus-notification
Merge into: lp://qastaging/qtmir
Diff against target: 65 lines (+11/-6)
3 files modified
src/modules/Unity/Application/application_manager.cpp (+6/-2)
src/modules/Unity/Application/dbuswindowstack.h (+2/-2)
tests/modules/ApplicationManager/application_manager_test.cpp (+3/-2)
To merge this branch: bzr merge lp://qastaging/~gerboland/qtmir/fix-unfocus-notification
Reviewer Review Type Date Requested Status
Daniel d'Andrada (community) Approve
Review via email: mp+228901@code.qastaging.launchpad.net

Commit message

AppMan: fix dbus GetWindowStack & GetAppFromPid APIs. On suspend, mark focused application unfocused, and reverse on resume.

Approved by: Daniel d'Andrada

Description of the change

AppMan: fix dbus GetWindowStack & GetAppFromPid APIs. On suspend, mark focused application unfocused, and reverse on resume.

 * Are there any related MPs required for this MP to build/function as expected? Please list.
N
 * Did you perform an exploratory manual test run of your code change and any related functionality?
Y
 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A

To post a comment you must log in.
Revision history for this message
Gerry Boland (gerboland) wrote :

Note the twin MR for this patch is here:
https://code.launchpad.net/~gerboland/unity-mir/unfocusOnGreeter/+merge/227940
(last commit needed to be added to qtmir)

Revision history for this message
Daniel d'Andrada (dandrader) wrote :

The commit message doesn't match what the diff actually does.

Revision history for this message
Daniel d'Andrada (dandrader) wrote :

Looks good.

review: Approve
225. By Gerry Boland

Merge trunk

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches