[Pkg-javascript-commits] [dojo] 82/87: Fixes #16833. Use removeNode on IE to avoid leaks when event handlers are on the node and removeChild everywhere else. Backport thru 1.6. !strict

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to annotated tag 1.7.5
in repository dojo.

commit 20e6337ce387c55664b67ed138be25d4f285be17
Author: Douglas Hays <doughays at dojotoolkit.org>
Date:   Wed Mar 27 22:19:40 2013 +0000

    Fixes #16833.  Use removeNode on IE to avoid leaks when event handlers are on the node and removeChild everywhere else.  Backport thru 1.6. !strict
    
    git-svn-id: http://svn.dojotoolkit.org/src/branches/1.7/dojo@31027 560b804f-0ae3-0310-86f3-f6aa0a117693
---
 dom-construct.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/dom-construct.js b/dom-construct.js
index 3617cdd..a86873c 100644
--- a/dom-construct.js
+++ b/dom-construct.js
@@ -367,7 +367,11 @@ define(["exports", "./_base/kernel", "./_base/sniff", "./_base/window", "./dom",
 			_empty(node);
 		}
 		if(parent){
-			parent.removeChild(node);
+			//>>excludeStart("webkitMobile", kwArgs.webkitMobile);
+			// removeNode(false) doesn't leak in IE 6+, but removeChild() and removeNode(true) are known to leak under IE 8- while 9+ is TBD
+			has("ie") ? node.removeNode(false) :
+			//>>excludeEnd("webkitMobile");
+				parent.removeChild(node);
 		}
 	}
 	exports.destroy = function destroy(/*DOMNode|String*/ node){

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/dojo.git