#8545 closed bug (fixed)

Opened March 16, 2011 08:22PM UTC

Closed May 16, 2012 06:04PM UTC

Last modified August 01, 2012 05:55PM UTC

Leak with events in IE

Reported by:	olau	Owned by:
Priority:	high	Milestone:	1.8
Component:	event	Version:	1.5.1
Keywords:		Cc:
Blocked by:		Blocking:

Description

The test case in issue 2123 is leaking for me in IE 6, 7 and 8 with jQuery 1.5.1. Here's a version that loops and setTimeouts to make the problem easier to see:

http://jsfiddle.net/En8Ue/

Just open task manager and watch it grow.

I was going to say that this is only a problem with custom events, but if I change 'pseudoEvent' to 'click', it does in fact still leak, although not as much.

Attachments (0)

Change History (11)

Changed March 17, 2011 06:59PM UTC by dmethvin comment:1

component:	unfiled → event
milestone:	1.next → 1.6
owner:	→ dmethvin
priority:	undecided → high
status:	new → assigned

I have been able to repro this and am looking at it.

Changed April 17, 2011 09:19PM UTC by john comment:2

priority:	high → blocker

Changed April 25, 2011 12:38AM UTC by dmethvin comment:3

milestone:	1.6 → 1.7
priority:	blocker → high

This is a horribly frustrating leak! It seems like there is some interaction with jsFiddle that prevents the test case from failing (leaking) unless it is loaded from a clean start. If you simply change the version of jQuery and press Run or reload it won't always start leaking.

That said, I don't think this is a blocker. I can seem to reproduce the behavior consistently all the way back to 1.3.2 in IE8 so if there is a leak it has been going on quite a while.

http://jsfiddle.net/En8Ue/7/

This test case is an extreme so perhaps in normal cases this leak isn't significant enough to cause problems. I walked through the code and it seems to be releasing things properly, but I'll leave it marked high and we can come back to it later.

Changed October 25, 2011 10:56AM UTC by anonymous comment:4

Can we have a status update on this one?

Changed October 25, 2011 12:54PM UTC by dmethvin comment:5

I've been through the code and can't find the leak. It still seems to be present in 1.7. Ideas welcome.

Changed November 11, 2011 01:54PM UTC by dmethvin comment:6

milestone:	1.7 → 1.next
owner:	dmethvin
status:	assigned → open

Changed May 16, 2012 06:04PM UTC by Oleg comment:7

resolution:	→ fixed
status:	open → closed

Fix #8545. Plug event handling memory leak in oldIE.

Changeset: 203a168980ece60213c87a10ac0ad294783d5563

Changed May 16, 2012 06:12PM UTC by dmethvin comment:8

milestone:	1.next → 1.8

Changed July 26, 2012 02:22AM UTC by anonymous comment:9

'change' event handler leaks in IE 7/8: http://jsfiddle.net/En8Ue/51/

Changed July 26, 2012 02:23AM UTC by anonymous comment:10

Replying to [comment:9 anonymous]: