Merge lp:~compiz-team/compiz/compiz.fix_1189369 into lp:compiz/0.9.10

Proposed by Sam Spilsbury on 2013-06-10
Status: Merged
Approved by: MC Return on 2013-06-11
Approved revision: 3738
Merged at revision: 3738
Proposed branch: lp:~compiz-team/compiz/compiz.fix_1189369
Merge into: lp:compiz/0.9.10
Diff against target: 27 lines (+16/-0)
1 file modified
plugins/opengl/src/window.cpp (+16/-0)
To merge this branch: bzr merge lp:~compiz-team/compiz/compiz.fix_1189369
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration 2013-06-10 Approve on 2013-06-11
MC Return 2013-06-10 Approve on 2013-06-10
Review via email: mp+168467@code.launchpad.net

This proposal supersedes a proposal from 2013-06-10.

Commit message

Immediately update the window matrices and regions if the number of
textures changed.

(LP: #1189369)

Description of the change

Immediately update the window matrices and regions if the number of
textures changed.

(LP: #1189369)

To post a comment you must log in.
MC Return (mc-return) wrote :

LGTM AFAICT ;)

review: Approve
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/opengl/src/window.cpp'
2--- plugins/opengl/src/window.cpp 2013-01-03 16:05:26 +0000
3+++ plugins/opengl/src/window.cpp 2013-06-10 14:47:33 +0000
4@@ -179,8 +179,24 @@
5 }
6 else
7 {
8+ bool immediatelyUpdateMatricesAndRegions =
9+ priv->textures.size () != textures.size ();
10+
11 priv->textures = textures;
12 priv->needsRebind = false;
13+
14+ /* If the number of textures changed, we should immediately
15+ * update the matrices and regions so that they are at least
16+ * initialized, but we'll queue another update just before
17+ * glPaint too in case the window moved or changed size */
18+ if (immediatelyUpdateMatricesAndRegions)
19+ {
20+ priv->setWindowMatrix ();
21+ priv->updateWindowRegions ();
22+
23+ priv->updateState |= PrivateGLWindow::UpdateMatrix |
24+ PrivateGLWindow::UpdateRegion;
25+ }
26 }
27 }
28

Subscribers

People subscribed via source and target branches

to all changes: