#8545 closed bug (fixed)
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:
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.
Change History (11)
comment:1 Changed 12 years ago by
Component: | unfiled → event |
---|---|
Milestone: | 1.next → 1.6 |
Owner: | set to dmethvin |
Priority: | undecided → high |
Status: | new → assigned |
comment:2 Changed 12 years ago by
Priority: | high → blocker |
---|
comment:3 Changed 12 years ago by
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.
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.
comment:5 Changed 12 years ago by
I've been through the code and can't find the leak. It still seems to be present in 1.7. Ideas welcome.
comment:6 Changed 12 years ago by
Milestone: | 1.7 → 1.next |
---|---|
Owner: | dmethvin deleted |
Status: | assigned → open |
comment:7 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | open → closed |
Fix #8545. Plug event handling memory leak in oldIE.
Changeset: 203a168980ece60213c87a10ac0ad294783d5563
comment:8 Changed 11 years ago by
Milestone: | 1.next → 1.8 |
---|
comment:9 follow-up: 10 Changed 11 years ago by
'change' event handler leaks in IE 7/8: http://jsfiddle.net/En8Ue/51/
comment:10 follow-up: 11 Changed 11 years ago by
Replying to anonymous:
'change' event handler leaks in IE 7/8: http://jsfiddle.net/En8Ue/51/
With the latest version of jquery.
comment:11 Changed 11 years ago by
Replying to anonymous:
Replying to anonymous:
'change' event handler leaks in IE 7/8: http://jsfiddle.net/En8Ue/51/
With the latest version of jquery.
Its very different leak, created ticket for it – http://bugs.jquery.com/ticket/12177
I have been able to repro this and am looking at it.