detachEvent is unnecessary since we're nulling div. Fixes #8873.

timmywil · timmywil · commit b8fc9d14a134 · 2011-05-25T15:08:37.000-04:00
diff --git a/src/support.js b/src/support.js
@@ -109,11 +109,10 @@ jQuery.support = (function() {
 	}
 
 	if ( !div.addEventListener && div.attachEvent && div.fireEvent ) {
-		div.attachEvent( "onclick", function click() {
+		div.attachEvent( "onclick", function() {
 			// Cloning a node shouldn't copy over any
 			// bound event handlers (IE does this)
 			support.noCloneEvent = false;
-			div.detachEvent( "onclick", click );
 		});
 		div.cloneNode( true ).fireEvent( "onclick" );
 	}

Original file line number	Diff line number	Diff line change
`@@ -109,11 +109,10 @@ jQuery.support = (function() {`
`109`	`109`	`}`
`110`	`110`
`111`	`111`	`if ( !div.addEventListener && div.attachEvent && div.fireEvent ) {`
`112`		`- div.attachEvent( "onclick", function click() {`
	`112`	`+ div.attachEvent( "onclick", function() {`
`113`	`113`	`// Cloning a node shouldn't copy over any`
`114`	`114`	`// bound event handlers (IE does this)`
`115`	`115`	`support.noCloneEvent = false;`
`116`		`- div.detachEvent( "onclick", click );`
`117`	`116`	`});`
`118`	`117`	`div.cloneNode( true ).fireEvent( "onclick" );`
`119`	`118`	`}`