Memory leaks when bind custom event in IE8

[fallingdust]

In jQuery, when binding a custom event, i.e., 'abc' in IE8, and then remove the element by calling remove(). attachEvent and detachEvent will be called. But actually detachEvent won't break the reference chain, the memory leaks...

Also, the following code leaks in IE8: function testLeak(){

var div = document.createElement('div'); div.innerHTML = new Array(1000).join(new Array(1000).join('x')); var handler = function(){ }; div.attachEvent('onabc', handler); div.detachEvent('onabc', handler);

} setInterval(testLeak, 1000);

[snover]

Please provide a link to a live test case demonstrating this issue.

[fallingdust]

Here is the test case on jsfiddle: ​http://jsfiddle.net/xPNb3/1/

[snover]

​No memory leak with standard event ​No memory leak if we call unbind first ​Revised test case that does not also load Mootools, and also just makes sure there is no weirdness going on with creating the string data via Array.join

[snover]

Retargeting due to severity of 1.4.3 regressions.

[snover]

Retargeting for 1.4.4 as per John’s request.

[snover]

Retargeting to next minor release.

[dmethvin]

Fix: ​https://github.com/jquery/jquery/pull/158

[Dave Methvin]

Ensure that the DOM element ref in an event handler is removed by cleanData to avoid an IE6/7/8 memory leak. Fixes #7054.

Changeset: fcf623786aeae20485e5253bd2b66c8758053646

[jitter]

Move fixed tickets to appropriate milestone

[dmethvin]

#12055 is a duplicate of this ticket.

[thorn.mailbox@…]

It isn't fixed. See the test case here: ​http://www.nesterovsky-bros.com/weblog/2013/02/10/jquerysTickets7054ClosedBugFixed.aspx

[anonymous]

+1, this isn't fixed, the issue is leftover anonymous functions and closure from event handling.