[Commits] r1102 - core/trunk/geoext/tests/lib/GeoExt/data

[commits at geoext.org]

Author: ahocevar
Date: 2009-06-17 16:19:27 +0200 (Wed, 17 Jun 2009)
New Revision: 1102

Modified:
   core/trunk/geoext/tests/lib/GeoExt/data/LayerStore.html
Log:
tschaub is right - the test will also show regressions without doing control stuff in the remove handler. (see #88)


Modified: core/trunk/geoext/tests/lib/GeoExt/data/LayerStore.html
===================================================================
--- core/trunk/geoext/tests/lib/GeoExt/data/LayerStore.html	2009-06-17 14:18:30 UTC (rev 1101)
+++ core/trunk/geoext/tests/lib/GeoExt/data/LayerStore.html	2009-06-17 14:19:27 UTC (rev 1102)
@@ -403,16 +403,9 @@
             });
             
             var count = 0;
-            // destroying and adding this.control to the map in the remove
-            // handler will cause exceptions when done while the map is being
-            // destroyed. This is the pattern that helped investigating #88.
-            this.control = new OpenLayers.Control();
             store.on("remove", function() {
                 count++;
-                this.control.destroy();
-                this.control = new OpenLayers.Control();
-                map.addControl(this.control);
-            }, this);
+            });
             
             map.removeLayer(a);
             map.destroy();