Throttle window movement
Bug #969108 reported by
Sam Spilsbury
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Compiz |
Invalid
|
Medium
|
Unassigned | ||
| Compiz Core |
Invalid
|
Medium
|
Unassigned | ||
Bug Description
See LP#898465 and LP#764330
Currently we process geometry updates as we receive motion events - this is impractical as we could receive a lot of motion events between paints and thus we are updating state when we don't really need to be. Throttle state updates and notification to repaints.
Related branches
lp:~smspillaz/compiz-core/compiz-core.fix_969108.2
- Daniel van Vugt: Pending requested
-
Diff: 1337 lines (+1127/-7)12 files modifiedinclude/core/screen.h (+1/-0)
plugins/CMakeLists.txt (+1/-0)
plugins/move/src/move.cpp (+48/-5)
plugins/move/src/move.h (+32/-2)
src/CMakeLists.txt (+5/-0)
src/queues/CMakeLists.txt (+61/-0)
src/queues/include/core/queues.h (+362/-0)
src/queues/src/queues.cpp (+115/-0)
src/queues/tests/CMakeLists.txt (+18/-0)
src/queues/tests/queues/src/test-queues.cpp (+237/-0)
src/queues/tests/test-queues.cpp (+34/-0)
src/queues/tests/test-queues.h (+213/-0)
- Daniel van Vugt: Pending requested
-
Diff: 1337 lines (+1127/-7)12 files modifiedinclude/core/screen.h (+1/-0)
plugins/CMakeLists.txt (+1/-0)
plugins/move/src/move.cpp (+48/-5)
plugins/move/src/move.h (+32/-2)
src/CMakeLists.txt (+5/-0)
src/queues/CMakeLists.txt (+61/-0)
src/queues/include/core/queues.h (+362/-0)
src/queues/src/queues.cpp (+115/-0)
src/queues/tests/CMakeLists.txt (+18/-0)
src/queues/tests/queues/src/test-queues.cpp (+237/-0)
src/queues/tests/test-queues.cpp (+34/-0)
src/queues/tests/test-queues.h (+213/-0)
lp:~vanvugt/compiz-core/lastMotionTime
- Sam Spilsbury: Disapprove
- Alan Griffiths: Approve
-
Diff: 75 lines (+17/-13)3 files modifiedmetadata/core.xml.in (+7/-0)
src/privatescreen.h (+1/-0)
src/screen.cpp (+9/-13)
| Changed in compiz-core: | |
| importance: | Undecided → Medium |
| assignee: | nobody → Sam Spilsbury (smspillaz) |
| milestone: | none → 0.9.7.6 |
| status: | New → Incomplete |
| status: | Incomplete → In Progress |
Revision history for this message
| Daniel van Vugt (vanvugt) wrote | #1 |
| Changed in compiz-core: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Daniel van Vugt (vanvugt) wrote | #2 |
| Changed in compiz-core: | |
| status: | Fix Committed → In Progress |
| Changed in compiz-core: | |
| milestone: | 0.9.7.6 → 0.9.7.8 |
| Changed in compiz-core: | |
| milestone: | 0.9.7.8 → 0.9.8.0 |
| Changed in compiz: | |
| assignee: | nobody → Sam Spilsbury (smspillaz) |
| importance: | Undecided → Medium |
| status: | New → In Progress |
| Changed in compiz: | |
| milestone: | none → 0.9.8.0 |
| Changed in compiz-core: | |
| milestone: | 0.9.8.0 → none |
Revision history for this message
| Sam Spilsbury (smspillaz) wrote | #3 |
| Changed in compiz: | |
| milestone: | 0.9.8.0 → none |
| assignee: | Sam Spilsbury (smspillaz) → nobody |
| status: | In Progress → Invalid |
| Changed in compiz-core: | |
| status: | In Progress → Invalid |
| assignee: | Sam Spilsbury (smspillaz) → nobody |
To post a comment you must log in.
Fix committed into lp:compiz-core at revision 3080