Merge lp:~vanvugt/compiz-plugins-main/fix-930192 into lp:compiz-plugins-main

Proposed by Daniel van Vugt on 2012-03-20
Status: Merged
Merged at revision: 25
Proposed branch: lp:~vanvugt/compiz-plugins-main/fix-930192
Merge into: lp:compiz-plugins-main
Diff against target: 19 lines (+5/-0)
1 file modified
animation/src/animation.cpp (+5/-0)
To merge this branch: bzr merge lp:~vanvugt/compiz-plugins-main/fix-930192
Reviewer Review Type Date Requested Status
Alan Griffiths 2012-03-20 Approve on 2012-03-20
Review via email: mp+98387@code.launchpad.net

Description of the change

Fix stuttering animations on large/fullscreen windows by using the higher
performance rendering method (full screen damage). (LP: #930192)

To post a comment you must log in.
Alan Griffiths (alan-griffiths) wrote :

Brutal

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'animation/src/animation.cpp'
2--- animation/src/animation.cpp 2012-01-23 10:23:25 +0000
3+++ animation/src/animation.cpp 2012-03-20 10:44:15 +0000
4@@ -901,10 +901,15 @@
5 void
6 PrivateAnimWindow::damageThisAndLastStepRegion ()
7 {
8+#ifdef COMPIZ_OPENGL_SWAPBUFFERS_ALWAYS // when LP: #901097 is fixed
9 // Find union of the regions for this step and last step
10 CompRegion totalRegionToDamage (mStepRegion + mLastStepRegion);
11
12 mPAScreen->cScreen->damageRegion (totalRegionToDamage);
13+#else
14+ // Ugly fix for LP: #930192 while LP: #901097 is not resolved
15+ mPAScreen->cScreen->damageScreen ();
16+#endif
17 }
18
19 CompOutput &

Subscribers

People subscribed via source and target branches