From 6dd302e5ce187d6db9266a7d6c0b94fd37fa46f4 Mon Sep 17 00:00:00 2001
From: Jonathan Matthew <jonathan@d14n.org>
Date: Fri, 6 Feb 2009 10:57:54 +1000
Subject: [PATCH] Bug 570583: handle tweens being overwritten during window
 effects

Since tweener can't do multiple tweens on the same property, it
calls an overwrite callback for any existing tweens when a new
tween is added for the same property.  Here we use the overwrite
callback to tell mutter that the effects are done.
---
 js/ui/windowManager.js | 76 ++++++++++++++++++++++++++++++------------
 1 file changed, 55 insertions(+), 21 deletions(-)

diff --git a/js/ui/windowManager.js b/js/ui/windowManager.js
index 77231f9d3..3d27eb4f0 100644
--- a/js/ui/windowManager.js
+++ b/js/ui/windowManager.js
@@ -87,6 +87,15 @@ WindowManager.prototype = {
         return true;
     },
 
+    _removeEffect : function(list, actor) {
+        let idx = list.indexOf(actor);
+        if (idx != -1) {
+            list.splice(idx, 1);
+            return true;
+        }
+        return false;
+    },
+
     _minimizeWindow : function(actor) {
         if (!this._shouldAnimate(actor)) {
             this._shellwm.completed_minimize(actor);
@@ -107,22 +116,29 @@ WindowManager.prototype = {
                            transition: "easeOutQuad",
                            onComplete: this._minimizeWindowDone,
                            onCompleteScope: this,
-                           onCompleteParams: [actor]
+                           onCompleteParams: [actor],
+                           onOverwrite: this._minimizeWindowOverwritten,
+                           onOverwriteScope: this,
+                           onOverwriteParams: [actor]
                          });
     },
 
     _minimizeWindowDone : function(actor) {
-        let idx = this._minimizing.indexOf(actor);
-        if (idx != -1) {
+        if (this._removeEffect(this._minimizing, actor)) {
             Tweener.removeTweens(actor);
             actor.set_scale(1.0, 1.0);
             actor.move_anchor_point_from_gravity(Clutter.Gravity.NORTH_WEST);
-            this._shellwm.completed_minimize(actor);
 
-            this._minimizing.splice(idx, 1);
+            this._shellwm.completed_minimize(actor);
         }
     },
-    
+
+    _minimizeWindowOverwritten : function(actor) {
+        if (this._removeEffect(this._minimizing, actor)) {
+            this._shellwm.completed_minimize(actor);
+        }
+    },
+
     _maximizeWindow : function(actor, target_x, target_y, target_width, target_height) {
         if (!this._shouldAnimate(actor)) {
             this._shellwm.completed_maximize(actor);
@@ -148,19 +164,25 @@ WindowManager.prototype = {
                            transition: "easeOutQuad",
                            onComplete: this._maximizeWindowDone,
                            onCompleteScope: this,
-                           onCompleteParams: [actor]
+                           onCompleteParams: [actor],
+                           onOverwrite: this._maximizeWindowOverwrite,
+                           onOverwriteScope: this,
+                           onOverwriteParams: [actor]
                          });
     },
 
     _maximizeWindowDone : function(actor) {
-        let idx = this._maximizing.indexOf(actor);
-        if (idx != -1) {
+        if (this._removeEffect(this._maximizing, actor)) {
             Tweener.removeTweens(actor);
             actor.set_scale(1.0, 1.0);
             actor.move_anchor_point_from_gravity(Clutter.Gravity.NORTH_WEST);
             this._shellwm.completed_maximize(actor);
-            
-            this._maximizing.splice(idx, 1);
+        }
+    },
+
+    _maximizeWindowOverwrite : function(actor) {
+        if (this._removeEffect(this._maximizing, actor)) {
+            this._shellwm.completed_maximize(actor);
         }
     },
 
@@ -190,19 +212,25 @@ WindowManager.prototype = {
                            transition: "easeOutQuad",
                            onComplete: this._mapWindowDone,
                            onCompleteScope: this,
-                           onCompleteParams: [actor]
+                           onCompleteParams: [actor],
+                           onOverwrite: this._mapWindowOverwrite,
+                           onOverwriteScope: this,
+                           onOverwriteParams: [actor]
                          });
     },
 
     _mapWindowDone : function(actor) {
-        let idx = this._mapping.indexOf(actor);
-        if (idx != -1) {
+        if (this._removeEffect(this._mapping, actor)) {
             Tweener.removeTweens(actor);
             actor.set_scale(1.0, 1.0);
             actor.move_anchor_point_from_gravity(Clutter.Gravity.NORTH_WEST);
             this._shellwm.completed_map(actor);
-            
-            this._mapping.splice(idx, 1);
+        }
+    },
+
+    _mapWindowOverwrite : function(actor) {
+        if (this._removeEffect(this._mapping, actor)) {
+            this._shellwm.completed_map(actor);
         }
     },
 
@@ -223,18 +251,24 @@ WindowManager.prototype = {
                            transition: "easeOutQuad",
                            onComplete: this._destroyWindowDone,
                            onCompleteScope: this,
-                           onCompleteParams: [actor]
+                           onCompleteParams: [actor],
+                           onOverwrite: this._destroyWindowOverwrite,
+                           onOverwriteScope: this,
+                           onOverwriteParams: [actor]
                          });
     },
     
     _destroyWindowDone : function(actor) {
-        let idx = this._destroying.indexOf(actor);
-        if (idx != -1) {
+        if (this._removeEffect(this._destroying, actor)) {
             this._shellwm.completed_destroy(actor);
             Tweener.removeTweens(actor);
             actor.set_scale(1.0, 1.0);
-            
-            this._mapping.splice(idx, 1);
+        }
+    },
+
+    _destroyWindowOverwrite : function(actor) {
+        if (this._removeEffect(this._destroying, actor)) {
+            this._shellwm.completed_destroy(actor);
         }
     },