Ticket #4693 (closed bug: worksforme)
Memory leak with framesets in modal dialog
| Reported by: | dberthia | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | |
| Component: | event | Version: | 1.3.2 |
| Keywords: | ie memory leak | Cc: | |
| Blocking: | Blocked by: |
Description
In IE6/7, jQuery 1.3.2 introduces a browser memory leak when opening a modal dialog window that contains a frameset. The leak is made worse if there are nested framesets in the popup window, and pages within the frameset include jQuery. See attached files to reproduce the problem.
Attachments
Change History
comment:1 Changed 4 years ago by dmethvin
- Priority changed from critical to major
- Milestone changed from 1.3.2 to 1.3.3
comment:3 Changed 4 years ago by dberthia
I'm heartened to see some activity on this. We had to remove jQuery from our modal dialogs because of this, which obviously seriously restricts us from a development standpoint.
comment:5 Changed 3 years ago by dberthia
I'm now disheartened so see 1.4 roll out and still no activity on this.
comment:6 Changed 3 years ago by re076
Try this fix to 1.4.2
add to line 67: FSP change DOMContentUnLoaded,
add to line 756: FSP change DOMContentUnLoaded = function() { Original
for ( var id in jQuery.cache ) {
if ( jQuery.cache[ id ].handle ) { Try/Catch is to handle iframes being unloaded, see #4280
try {
jQuery.event.remove( jQuery.cache[ id ].handle.elem );
} catch(e) {}
}
}
FSP extension window.detachEvent( "onload", jQuery.ready ); window.detachEvent( "onunload", DOMContentUnLoaded ); window.jQuery = window.$ = jQuery = null;
}; FSP change end
change line 2627: if ( window.attachEvent && !window.addEventListener ) {
FSP change
window.attachEvent("onunload", DOMContentUnLoaded );
}
comment:7 Changed 3 years ago by SlexAxton
- Priority changed from major to low
- Status changed from new to open
- Milestone changed from 1.4 to 1.5
Seems to still exist in 1.4.x - marking as valid.
comment:8 Changed 2 years ago by john
- Status changed from open to closed
- Resolution set to worksforme
- Component changed from core to event
I believe that this has been resolved. Please re-open if that is no longer the case.
comment:9 Changed 2 years ago by alexgalp
I'm checked the attached test case on ie7 and 6 with jQuery 1.6.1 and the leak is still occurs.
comment:10 Changed 2 years ago by anonymous
The bug still happens! is it a dead end?
comment:11 Changed 2 years ago by rwaldron
#9462 is a duplicate of this ticket.
comment:12 follow-up: ↓ 13 Changed 2 years ago by rwaldron
#9471 is a duplicate of this ticket.
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.


Unzip and launch index.htm in IE6. Use Task Manager to monitor as you open/close the popup window.