Opened 12 years ago

Closed 12 years ago

#11608 closed defect (fixed)

popup iframe leak

Reported by: bill Owned by: bill
Priority: high Milestone: 1.5.1
Component: Dijit Version: 1.5
Keywords: Cc: Douglas Hays, Katie Vance
Blocked By: Blocking:

Description

There's an iframe leak when repeatedly opening/closing popups.

Seen in:

  • test_ComboBox.html (1.5 branch, 1.4 branch)
  • test_TabContainer.html (only trunk, b/c of [22703])
  • test_DateTextBox.html (1.5 branch. 1.4 branch leaks a DIV but no iframe)

Used sIeve/IE6, based on the "show in use" button. (This was originally filed under #11590, from Doug)

Main problem seems to be that dijit.popup.open() creates a !BackgroundIFrame class (with an associated iframe), and neither the class nor the iframe are ever released.

Change History (3)

comment:1 Changed 12 years ago by Douglas Hays

Cc: Katie Vance added

comment:2 Changed 12 years ago by bill

(In [22842]) Fix iframe leak when popup widget is destroyed. Changed moveOffScreen() to take a widget as it's argument, rather than a DOMNode (but still supporting the old signature for backwards compatibility).

Checking the fix into trunk first, and will backport to 1.5/ if everything seems OK after a while.

comment:3 Changed 12 years ago by bill

Resolution: → fixed
Status: new → closed

(In [22888]) merge iframe leak fixes to 1.5 branch, fixes #11608 !strict

Note: See TracTickets for help on using tickets.