Merge lp://staging/~smspillaz/compiz-core/compiz-core.decor_synchronization into lp://staging/compiz-core/0.9.8
Status: | Merged |
---|---|
Approved by: | Sam Spilsbury |
Approved revision: | 3024 |
Merged at revision: | 3131 |
Proposed branch: | lp://staging/~smspillaz/compiz-core/compiz-core.decor_synchronization |
Merge into: | lp://staging/compiz-core/0.9.8 |
Diff against target: |
1457 lines (+849/-66) 15 files modified
gtk/window-decorator/decorator.c (+48/-24) gtk/window-decorator/events.c (+12/-1) gtk/window-decorator/gtk-window-decorator.c (+9/-0) gtk/window-decorator/gtk-window-decorator.h (+8/-0) gtk/window-decorator/wnck.c (+13/-11) include/decoration.h (+18/-0) libdecoration/decoration.c (+86/-0) plugins/decor/CMakeLists.txt (+3/-1) plugins/decor/src/decor.cpp (+99/-22) plugins/decor/src/decor.h (+35/-7) plugins/decor/src/pixmap-requests/CMakeLists.txt (+62/-0) plugins/decor/src/pixmap-requests/include/pixmap-requests.h (+188/-0) plugins/decor/src/pixmap-requests/src/pixmap-requests.cpp (+93/-0) plugins/decor/src/pixmap-requests/tests/CMakeLists.txt (+15/-0) plugins/decor/src/pixmap-requests/tests/pixmap-requests/src/test-decor-pixmap-requests.cpp (+160/-0) |
To merge this branch: | bzr merge lp://staging/~smspillaz/compiz-core/compiz-core.decor_synchronization |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel van Vugt | Needs Resubmitting | ||
Alan Griffiths | Pending | ||
Review via email:
|
This proposal supersedes a proposal from 2012-04-25.
Description of the change
Add synchronization primitives to the decoration protocol so that there isn't a race where we bind a texture that's being freed.
Things changed
* update_
* decor_post_pending : tells the decorated process that a new decoration is pending
* decor_post_request : tells the decorator that a new pixmap can be drawn to
* decor_post_
I also did some necessary refactoring so we could get this under test, namely:
DecorPixmap class (implements DecorPixmapInte
X11DecorPixmapR
X11DecorPixmapR
X11PixmapDeletor class (implements DecorPixmapDele
DecorationListF
DecorationInter
Tests included (Google Mock)
I haven't reviewed this, but please fix and/or resubmit to resolve the conflicts.