Merge lp:~ivenvd/compiz/compiz.fix_1189374 into lp:compiz/0.9.10

[Iven Hsu]

Opacify: Only dim the windows above the active window.(LP: #1189374)

See: http://bazaar.launchpad.net/~compiz-team/compiz/0.9.10/revision/3727

[MC Return (mc-return)]

Yes, this is correct.

Thanks for fixing it. +1.
Could you please add a commit message so we can merge this ?

I noticed another bug though:
The "Only increase opacity if a window is blocking" option in CCSM is not even implemented. But it has nothing to do with this fix and the regression I introduced.

review: Approve

[Sam Spilsbury (smspillaz)]

review: Approve

[Sam Spilsbury (smspillaz)]

(A word of warning about why code changes without tests are a bad idea).

[PS Jenkins bot (ps-jenkins)]

review: Approve (continuous-integration)

=== modified file 'plugins/opacify/src/opacify.cpp'
--- plugins/opacify/src/opacify.cpp	2013-05-28 08:21:49 +0000
+++ plugins/opacify/src/opacify.cpp	2013-06-10 08:51:28 +0000
@@ -156,9 +156,10 @@
 	    continue;
 	}
 
-	if (!flag				    &&
-	    !optionGetWindowMatch ().evaluate (w)   &&
-	    (!w->isViewable () || w->minimized ()))
+	if (!flag				    ||
+	    !optionGetWindowMatch ().evaluate (w)   ||
+	    !w->isViewable ()			    ||
+	    w->minimized ())
 	    continue;
 
 	intersect = w->region ().intersected (fRegion);