Bug Tracker

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#9294 closed bug (fixed)

Memory leaks in IE(6/7/8) when loading frame, which is in a frameset and includes latest jQuery (v.1.5.1 and above) with solution

Reported by: romero83@… Owned by:
Priority: high Milestone: 1.next
Component: support Version: 1.6.1
Keywords: Cc:
Blocked by: Blocking:

Description

Ok. So most probably I found the problem for this memory leak issue. Let's see the latest Jquery 1.6.1 source code. The problematic code line is 1172, which looks like this:

input = div.getElementsByTagName( "input" )[ 0 ];

I don't know why, but if you add this statement into a variable, then IE do some kind of memory issue. After I commented out that line, no more memory leak appeared in IE7.

I did a workaround in version 1.5.1 and it worked.

See http://bugs.jquery.com/ticket/8863 for more details.

I hope this helps to fix this problem.

Change History (5)

comment:1 Changed 12 years ago by Timmy Willison

Cc: addyosmani added
Component: unfiledsupport
Priority: undecidedhigh
Status: newopen

It would be great if that fixes the issue. CC addy.

comment:2 Changed 12 years ago by Booab

Thanks romero83 for the input but it doesn't solve the problem for me :-(
Also jQuery 1.4.4 is not working fine for me I also get a memory leak of about 1Mb per load.
Any other ideas or a solution coming up from jQueries side?

comment:3 Changed 12 years ago by Timmy Willison

Resolution: fixed
Status: openclosed

Null created elements in support to avoid leaks in IE. Tested IE6-8. Leaks are contained to the byte. Fixes #9294.

Changeset: 657b197c193335899703fc158b153e2475cca539

comment:4 Changed 12 years ago by Booab

Thanks for your proposed solution but unfortunately it's not solving the problem in IE6 (still about 1Mb of leak)

comment:5 Changed 12 years ago by Timmy Willison

Cc: addyosmani removed

@Booab. According to my tests, there are no leaks in the support module in IE6. Are you sure you are testing against the git version of jQuery? If so, please provide a test case so that we can confirm.

Note: See TracTickets for help on using tickets.