Erik and I discussed this, and the destroy() here actually does the wrong thing.

the innerHTML still prevents the memory leak, but unregistering in this way
doesn't do the right thing. remove it. 


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

lib/OpenLayers/Icon.js

@@ -52,7 +52,6 @@ OpenLayers.Icon.prototype = {
     },
     
     destroy: function() {
-        OpenLayers.Event.stopObserving(this.imageDiv.firstChild);
         this.imageDiv.innerHTML = "";
         this.imageDiv = null;
     },