Merge lp:~3v1n0/unity/spread-title-flicker-fix-7.1 into lp:unity

[Marco Trevisan (Treviño)]

PluginAdapter: set the spread window state before spreading or the callback will get a wrong state

Backporting lp:~3v1n0/unity/spread-title-flicker-fix/+merge/190632

[Christopher Townsend (townsend)]

LGTM. +1

review: Approve

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:3565
http://jenkins.qa.ubuntu.com/job/unity-ci/437/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-saucy-amd64-ci/326
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-saucy-armhf-ci/324
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-saucy-i386-ci/325

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/unity-ci/437/rebuild

review: Approve (continuous-integration)

[Christopher Townsend (townsend)]

Marco,

Ugh, didn't notice that this MP was targeted to lp:unity instead of lp:unity/7.1. Need to fix that:)

[Marco Trevisan (Treviño) (3v1n0)]

2013/10/11 Christopher Townsend <email address hidden>:
> Marco,
>
> Ugh, didn't notice that this MP was targeted to lp:unity instead of lp:unity/7.1. Need to fix that:)

Yes, me as well :P
Reproposed as https://code.launchpad.net/~3v1n0/unity/spread-title-flicker-fix-7.1/+merge/190766

=== modified file 'unity-shared/PluginAdapter.cpp'
--- unity-shared/PluginAdapter.cpp	2013-09-14 09:56:10 +0000
+++ unity-shared/PluginAdapter.cpp	2013-10-11 17:42:16 +0000
@@ -972,9 +972,9 @@
   std::size_t num_windows = windows.size();
   if (num_windows > 1 || (force && num_windows))
   {
+    _spread_windows_state = true;
     std::string const& match = MatchStringForXids(windows);
     InitiateScale(match, state);
-    _spread_windows_state = true;
     return true;
   }
   return false;