Merge lp:~compiz-team/compiz/compiz.fix_1110138 into lp:compiz/0.9.9

[Sam Spilsbury]

Make sure to update the frame extents when clients ask us to (LP: #1110138)

Make sure to update the frame extents when clients ask us to (LP: #1110138)

Unfortunately I can't get this under test in a way that doesn't involve invasive code changes. I don't have time to do those right now.

[MC Return (mc-return)]

LGTM. +1.
Have compiled and tested this with Emerald, but not with Java apps.

review: Approve

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:3555
http://jenkins.qa.ubuntu.com/job/compiz-ci/6/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/compiz-clang-ci/./build=pbuilder,distribution=quantal,flavor=amd64/44/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/compiz-gles-ci/./build=pbuilder,distribution=quantal,flavor=amd64/43/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/compiz-pbuilder/./build=pbuilder,distribution=quantal,flavor=amd64/395/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/compiz-ci/6//rebuild/?

review: Approve (continuous-integration)

[Andrea Azzarone (azzar1)]

review: Approve

=== modified file 'plugins/decor/src/decor.cpp'
--- plugins/decor/src/decor.cpp	2013-02-03 18:19:21 +0000
+++ plugins/decor/src/decor.cpp	2013-02-10 08:47:22 +0000
@@ -2367,7 +2367,14 @@
 	    {
 		w = screen->findWindow (event->xclient.window);
 		if (w)
-		    DecorWindow::get (w)->update (true);
+		{
+		    DecorWindow *dw = DecorWindow::get (w);
+
+		    /* Set the frameExtentsRequested flag so that we know to
+		     * at least update _NET_WM_FRAME_EXTENTS (LP: #1110138) */
+		    dw->frameExtentsRequested = true;
+		    dw->update (true);
+		}
 	    }
 	    /* A decoration is pending creation, allow it to be created */
 	    if (event->xclient.message_type == decorPendingAtom)