Changeset 4100

Timestamp:

08/29/07 00:21:17 (1 year ago)

Author:

crschmidt

Message:

Properly destroy OpenLayers.Handler.MouseWheel, patch from fredj. Added
tests for it as well. (Closes #935)

Files:

• trunk/openlayers/lib/OpenLayers/Handler/MouseWheel.js (modified) (1 diff)
• trunk/openlayers/tests/Handler/test_MouseWheel.html (modified) (1 diff)

trunk/openlayers/lib/OpenLayers/Handler/MouseWheel.js

@@ -47 +47 @@
-     */    
-    destroy: function() {
-this.deactivate(
+OpenLayers.Handler.prototype.destroy.apply(this, arguments
-        this.wheelListener = null;
-        OpenLayers.Handler.prototype.destroy.apply(this, arguments);
-    },
-

trunk/openlayers/tests/Handler/test_MouseWheel.html

@@ -118 +118 @@
-             "deactivate returns true if the handler was active already");
-    }
+    function test_handler_MouseWheel_destroy(t) {
+        t.plan(1);
+        var control = new OpenLayers.Control();
+        var handler = new OpenLayers.Handler.MouseWheel(control);
+        handler.deactivate = function() { 
+            t.ok(true, "Deactivate called one time.");
+        }    
+        handler.destroy();
+    }
-
-