Merge lp:~brandontschaefer/unity/ignore-override-redirect-in-window-on-top into lp:unity
Proposed by
Brandon Schaefer
Status: | Merged |
---|---|
Approved by: | Marco Trevisan (Treviño) |
Approved revision: | no longer in the source branch. |
Merged at revision: | 2745 |
Proposed branch: | lp:~brandontschaefer/unity/ignore-override-redirect-in-window-on-top |
Merge into: | lp:unity |
Diff against target: |
127 lines (+54/-30) 3 files modified
tests/autopilot/unity/tests/launcher/test_icon_behavior.py (+27/-0) unity-shared/PluginAdapter.h (+2/-0) unity-shared/PluginAdapterCompiz.cpp (+25/-30) |
To merge this branch: | bzr merge lp:~brandontschaefer/unity/ignore-override-redirect-in-window-on-top |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Marco Trevisan (Treviño) | Approve | ||
Review via email: mp+124512@code.launchpad.net |
Commit message
Ignore windows that have Override Redirect set to true when checking if a window is on top.
Description of the change
=== Problem ===
The Ibus language bar is a gtk.WINDOW_POPUP which means it is ignored by the WM for stacking. This means it was staying on the top of all the windows, while not being able to tell if it was or not. In PluginAdapterCo
=== Fix ===
Ignore any window that has override redirect set to true. I also re-factored the code a bit. Instead of going through all the siblings of a window we start that top of the stacking and grab the first valid one.
=== Test ===
AP test coming
To post a comment you must log in.
I also could not reproduce this issue anymore:
59 - /* FIXME: This should be included by the above defaultViewport() check,
60 - * but it doesn't seem to work correctly when there's only one workspace
61 - * enabled, so please drop the line above when bug #996604 is fixed in
62 - * Compiz. */
If anyone is able to double check that as well that would be awesome :)