Merge lp://qastaging/~brandontschaefer/compiz/lp.1034616-fix into lp://qastaging/compiz/0.9.9
Status: | Merged |
---|---|
Approved by: | Sam Spilsbury |
Approved revision: | 3624 |
Merged at revision: | 3626 |
Proposed branch: | lp://qastaging/~brandontschaefer/compiz/lp.1034616-fix |
Merge into: | lp://qastaging/compiz/0.9.9 |
Diff against target: |
30 lines (+7/-2) 2 files modified
src/privatescreen.h (+5/-0) src/window.cpp (+2/-2) |
To merge this branch: | bzr merge lp://qastaging/~brandontschaefer/compiz/lp.1034616-fix |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Sam Spilsbury | Approve | ||
PS Jenkins bot | continuous-integration | Pending | |
Review via email:
|
Commit message
Move CompScreenImpl:
(LP: #1034616)
Description of the change
=== Problem ===
focusTopMostWin
=== Fix ===
Switch over to using the serverWindows list.
=== Test ===
Theres an AP test in unity that caught this problem.
As discussed, this is the correct way of doing it. Thanks.
It might be possible to throw a system test into the mix too - now that we have testhelper, we can write AP-like tests for compiz that run atop of xorg-gtest. Have a look into that if you get the chance.