Merge lp://staging/~3v1n0/unity/grab-check-enforce into lp://staging/unity

Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Brandon Schaefer
Approved revision: no longer in the source branch.
Merged at revision: 3815
Proposed branch: lp://staging/~3v1n0/unity/grab-check-enforce
Merge into: lp://staging/unity
Diff against target: 42 lines (+17/-3)
2 files modified
lockscreen/LockScreenController.cpp (+2/-2)
unity-shared/PluginAdapter.cpp (+15/-1)
To merge this branch: bzr merge lp://staging/~3v1n0/unity/grab-check-enforce
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Brandon Schaefer (community) Approve
Review via email: mp+219401@code.staging.launchpad.net

Commit message

PluginAdapter: try to grab the screen to make sure that the screen is not grabbed

FocusIn event with NotifyGrab mode is not always triggered by X when a window
takes the grab, then we need to ensure that there's no grab by trying to get one.
We only care about keyboard here, since pointer grabs don't affect us in general.

To post a comment you must log in.
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

LGTM

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

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.