Bug Tracker

#8287 closed bug (invalid)

Opened February 15, 2011 06:48PM UTC

Closed March 02, 2011 07:48AM UTC

Last modified March 14, 2012 12:21PM UTC

Elminate memory leaks reported by IEJSLeaksDetector.exe

Reported by: ss@technicalpursuit.com Owned by: ss@technicalpursuit.com
Priority: undecided Milestone: 1.next
Component: core Version: 1.5
Keywords: Cc:
Blocked by: Blocking:
Description

The IE leak detector identifies two areas where JQuery apparently fails to remove onload/onunload handlers:

in the bindReady function, replaced window.attachEvent("onload" jQuery.ready) with:

window.attachEvent( "onload", function() {
    window.detachEvent("onload", arguments.callee);
    jQuery.ready();
});

in the // Prevent memory leaks in IE section, before the onunload function close added:

window.detachEvent('onunload', arguments.callee);
window.$ = null;
window.jQuery = null;
Attachments (0)
Change History (2)

Changed February 15, 2011 09:54PM UTC by jitter comment:1

component: unfiled → core
description: The IE leak detector identifies two areas where JQuery apparently fails to remove onload/onunload handlers: \ \ in the bindReady function, replaced window.attachEvent("onload" jQuery.ready) with: \ \ window.attachEvent( "onload", function() { \ window.detachEvent("onload", arguments.callee); \ jQuery.ready(); \ }); \ \ \ in the // Prevent memory leaks in IE section, before the onunload function close added: \ \ window.detachEvent('onunload', arguments.callee); \ window.$ = null; \ window.jQuery = null; \ \ → The IE leak detector identifies two areas where JQuery apparently fails to remove onload/onunload handlers: \ \ in the bindReady function, replaced `window.attachEvent("onload" jQuery.ready)` with: \ {{{ \ window.attachEvent( "onload", function() { \ window.detachEvent("onload", arguments.callee); \ jQuery.ready(); \ }); \ }}} \ \ \ in the `// Prevent memory leaks in IE section`, before the onunload function close added: \ {{{ \ window.detachEvent('onunload', arguments.callee); \ window.$ = null; \ window.jQuery = null; \ }}}
owner: → ss@technicalpursuit.com
status: new → pending

I tried a simple page with the exe found here . Didn't show any leaks for me with 1.5 .

Please submit a reduced test case, which reproduces the issue you are experiencing, on http://jsfiddle.net. So that we can investigate this issue further. Also make sure to read the link given below, in order to provide a most useful bug report.

How to report bugs

Changed March 02, 2011 07:48AM UTC by trac-o-bot comment:2

resolution: → invalid
status: pending → closed

Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!