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

Proposed by Daniel van Vugt
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 Approve
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.
Revision history for this message
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
=== modified file 'animation/src/animation.cpp'
--- animation/src/animation.cpp 2012-01-23 10:23:25 +0000
+++ animation/src/animation.cpp 2012-03-20 10:44:15 +0000
@@ -901,10 +901,15 @@
901void901void
902PrivateAnimWindow::damageThisAndLastStepRegion ()902PrivateAnimWindow::damageThisAndLastStepRegion ()
903{903{
904#ifdef COMPIZ_OPENGL_SWAPBUFFERS_ALWAYS // when LP: #901097 is fixed
904 // Find union of the regions for this step and last step905 // Find union of the regions for this step and last step
905 CompRegion totalRegionToDamage (mStepRegion + mLastStepRegion);906 CompRegion totalRegionToDamage (mStepRegion + mLastStepRegion);
906907
907 mPAScreen->cScreen->damageRegion (totalRegionToDamage);908 mPAScreen->cScreen->damageRegion (totalRegionToDamage);
909#else
910 // Ugly fix for LP: #930192 while LP: #901097 is not resolved
911 mPAScreen->cScreen->damageScreen ();
912#endif
908}913}
909914
910CompOutput &915CompOutput &

Subscribers

People subscribed via source and target branches