Remove Icon-specific code from Markers Layer, patch by meek,me,

r=euzuro.  (Closes #818)


git-svn-id: http://svn.openlayers.org/trunk/openlayers@9233 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf

lib/OpenLayers/Icon.js

@@ -78,6 +78,9 @@ OpenLayers.Icon = OpenLayers.Class({
      * references and memory leaks
      */
     destroy: function() {
+        // erase any drawn elements
+        this.erase();
+
         OpenLayers.Event.stopObservingElement(this.imageDiv.firstChild); 
         this.imageDiv.innerHTML = "";
         this.imageDiv = null;
@@ -143,6 +146,16 @@ OpenLayers.Icon = OpenLayers.Class({
         return this.imageDiv;
     }, 
 
+    /** 
+     * Method: erase
+     * Erase the underlying image element.
+     *
+     */
+    erase: function() {
+        if (this.imageDiv != null && this.imageDiv.parentNode != null) {
+            OpenLayers.Element.remove(this.imageDiv);
+        }
+    }, 
     
     /** 
      * Method: setOpacity

lib/OpenLayers/Layer/Markers.js

@@ -122,10 +122,7 @@ OpenLayers.Layer.Markers = OpenLayers.Class(OpenLayers.Layer, {
     removeMarker: function(marker) {
         if (this.markers && this.markers.length) {
             OpenLayers.Util.removeItem(this.markers, marker);
-            if ((marker.icon != null) && (marker.icon.imageDiv != null) &&
+            marker.erase();
-                (marker.icon.imageDiv.parentNode == this.div) ) {
-                this.div.removeChild(marker.icon.imageDiv);    
-            }
         }
     },
 

lib/OpenLayers/Marker.js

@@ -89,6 +89,9 @@ OpenLayers.Marker = OpenLayers.Class({
      * know which layer it is attached to.)
      */
     destroy: function() {
+        // erase any drawn features
+        this.erase();
+
         this.map = null;
 
         this.events.destroy();
@@ -115,6 +118,16 @@ OpenLayers.Marker = OpenLayers.Class({
         return this.icon.draw(px);
     }, 
 
+    /** 
+    * Method: erase
+    * Erases any drawn elements for this marker.
+    */
+    erase: function() {
+        if (this.icon != null) {
+            this.icon.erase();
+        }
+    }, 
+
     /**
     * Method: moveTo
     * Move the marker to the new location.

tests/Layer/Markers.html

@@ -23,7 +23,7 @@
         t.eq( layer.markers.length, 1, "addLayer adds marker to layer." );
     }
     function test_addMarker_removeMarker (t) {
-        t.plan( 4 );
+        t.plan( 6 );
 
         var map = new OpenLayers.Map('map');
         var baseLayer = new OpenLayers.Layer.WMS("Test Layer", 
@@ -38,6 +38,8 @@
         t.ok(  marker.icon.imageDiv.parentNode == layer.div, "addMarker adds marker image node into layer node." ); 
         layer.removeMarker(marker);
         t.ok(  marker.icon.imageDiv.parentNode != layer.div, "removeMarker removes marker image node from layer node." ); 
+        layer.removeMarker(marker);
+        t.ok(true, "Removing marker twice does not fail.");
         layer.addMarker(marker);
         t.ok(  marker.icon.imageDiv.parentNode == layer.div, "addMarker adds marker image node into layer node." ); 
 
@@ -45,6 +47,12 @@
         layer.removeMarker(marker);
         t.ok(true, "removing marker when no markers present does not script error");
 
+        var l = new OpenLayers.Layer.Markers();
+        var marker = new OpenLayers.Marker(new OpenLayers.LonLat(5,40));
+        l.addMarker(marker);
+        l.removeMarker(marker);
+        t.ok(true, "Removing marker when layer not added to map does not fail.");
+
     }
     
     function test_markerMovement(t) {