Skip to main content

Bug Tracker

Side navigation

#3962 closed bug (fixed)

Opened January 22, 2009 07:36PM UTC

Closed February 07, 2009 04:58PM UTC

$.getScript leaks in IE

Reported by: john Owned by: john
Priority: major Milestone: 1.3.2
Component: ajax Version: 1.3.1
Keywords: Cc:
Blocked by: Blocking:
Description

I've found and fixed a small memory leak in the jQuery 1.2.6 codebase

that pertains to the getScript ajax methods. (around line 2670 in

jQuery 1.2.6)

the new code is as follows - I added

script.onload = script.onreadystatechange = null;

right before the head.removeChild( script ); call to help IE collect

its garbage.

// Attach handlers for all browsers
script.onload = script.onreadystatechange = function(){
       if ( !done && (!this.readyState ||
                       this.readyState == "loaded" || this.readyState == "complete") ) {
               done = true;
               success();
               complete();
               // handle memory leak in IE
               script.onload = script.onreadystatechange = null;
               head.removeChild( script );
       }
};
Attachments (0)
Change History (2)

Changed January 22, 2009 07:37PM UTC by john comment:1

owner: → john

Changed February 07, 2009 04:58PM UTC by john comment:2

resolution: → fixed
status: newclosed

Fixed in SVN rev [6179].