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@freemail.hu	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.

Attachments (0)

Change History (5)

Changed May 16, 2011 02:34PM UTC by timmywil comment:1

cc:	→ addyosmani
component:	unfiled → support
priority:	undecided → high
status:	new → open

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

Changed May 19, 2011 03:40PM UTC by Booab comment:2

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?

Changed May 25, 2011 06:31PM UTC by timmywil comment:3

resolution:	→ fixed
status:	open → closed

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

Changeset: 657b197c193335899703fc158b153e2475cca539

Changed May 27, 2011 08:41AM UTC by Booab comment:4

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

Changed May 27, 2011 03:08PM UTC by timmywil comment:5

cc:	addyosmani

@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.

Side navigation

#9294 closed bug (fixed)