Bug Tracker

Opened 17 years ago

Closed 17 years ago

#225 closed bug (fixed)

[PATCH] IE Memory Leaks from event system

Reported by: [email protected] Owned by:
Priority: critical Milestone:
Component: event Version:
Keywords: Cc: [email protected]
Blocked by: Blocking:

Description

Currently jQuery does not take care of the well known memory leak from adding but not removing event handlers in IE. This is a patch to add this in.

The diff: http://brandonaaron.net/jquery/memoryleak/event_leak_fix.diff The example without the fix: http://brandonaaron.net/jquery/memoryleak/event_leak.html The example with the fix: http://brandonaaron.net/jquery/memoryleak/event_leak_fix.html The blog entry: http://brandonaaron.net/articles/2006/09/30/fixing-jquerys-memory-leak

Change History (4)

comment:1 Changed 17 years ago by joern

Resolution: fixed
Status: newclosed

Added to SVN. Good job Brandon!

comment:2 Changed 17 years ago by danyalex

The hotfix removes the custom unload events also. It's better to check if type == 'unload' before removing the event.

comment:3 Changed 17 years ago by brandon.aaro

Resolution: fixed
Status: closedreopened

danyalex is correct. Here is an updated patch and example.

The example (you should get an alert onunload): http://brandonaaron.net/jquery/memoryleak2/event_leak_fix.html The diff: http://brandonaaron.net/jquery/memoryleak2/event_leak_fix.diff

Also this patch fixes a memory leak in Firefox related to the ready event. You can verify using the leak monitor extension vs this unfixed page: http://brandonaaron.net/jquery/memoryleak2/event_leak.html

comment:4 Changed 17 years ago by joern

Resolution: fixed
Status: reopenedclosed

I added it to SVN. Just reopen this ticket if there are other issues...

Note: See TracTickets for help on using tickets.