Changeset 9392 for trunk/Xul_XedLib/chrome/content/scenarixedlib/core/xed.js

Timestamp:

07/24/07 14:47:27 (5 years ago)

Author:

sys

Message:

memLeaks : ajout de la méthode destroy() pour compenser l'absence d'appel des destructor des XBL si les éléments ont été extraits du Document.

File:

• trunk/Xul_XedLib/chrome/content/scenarixedlib/core/xed.js (modified) (4 diffs)

trunk/Xul_XedLib/chrome/content/scenarixedlib/core/xed.js

@@ -394 +394 @@
                                 return (sPropertyName in pNode) ? NodeFilter.FILTER_ACCEPT : NodeFilter.FILTER_SKIP;
                         }, false);
-                return vTw.firstChild();
+                var vRes = vTw.firstChild();
+                vTw = null; //memLeak
+                return vRes;
         }
         return null;
@@ -1291 +1293 @@
                 }
                 vEditor.setFocusFrom(vFirstNode);
-        }
+                delete this.fFrag;
+        }
+}
+XEDInsertHisto.prototype.destroy = function(){
+        if(this.fFrag) xed.destroyBindings(this.fFrag);
 }
 
@@ -1329 +1335 @@
 }
 
+XEDReplaceHisto.prototype.destroy = function(){
+        xed.destroyBindings(this.fOldBind.parentNode ? this.fNewBind : this.fOldBind);
+}
 
 //**************************************************************************************
@@ -1716 +1725 @@
         return vS+"]";
 }
+
+xed.destroyBindings = function(pNode) {
+        if(!pNode) return;
+        function callDestroy(pNode){
+                try {
+                        if("destroy" in pNode) pNode.destroy();
+                } catch(e){xed.debug("destroyNode failed : "+pNode.localName+"\n"+e);}
+                return NodeFilter.FILTER_ACCEPT;
+        }
+        callDestroy(pNode);
+        var vTw = pNode.ownerDocument.createTreeWalker(pNode, NodeFilter.SHOW_ELEMENT, callDestroy, false);
+        while(vTw.nextNode());
+        vTw = null; //memLeak
+}
+
 /** Protège les -- par des -~- et plus généralement -(X)~- par -(X+)~- pour les commentaires imbriqués. */
 xed.escapeComment = function(pText){