Opened 12 years ago
Closed 12 years ago
#7206 closed defect (fixed)
[patch] [cla] TabContainer: closable tab pane leaks dom nodes when closed.
| Reported by: | afryer | Owned by: | bill |
|---|---|---|---|
| Priority: | high | Milestone: | 1.2 |
| Component: | Dijit | Version: | 1.1.1 |
| Keywords: | dom node leak TabContainer StackContainer addChild closable | Cc: | |
| Blocked By: | Blocking: |
Description
When a closable tab pane is created, it instantiates a menu and a menu item. When the tab pane is closed, the menu is destroyed using the destroy() method. The menuItem is not destroyed and is leaked.
I also see dojo.connect calls made on the button and menu widgets that don't have corresponding dojo.disconnect calls when the tab pane is closed.
Have attached a test case and a patch file to fix the leak.
Attachments (2)
Change History (6)
Changed 12 years ago by
| Attachment: | test_TabContainer_Programmatic_leak.html added |
|---|
Changed 12 years ago by
| Attachment: | StackContainer_patch.txt added |
|---|
patch for StackContainer?.js that fixes closable child leak
comment:1 Changed 12 years ago by
| Milestone: | tbd → 1.2 |
|---|---|
| Owner: | set to bill |
| Summary: | closable tab pane leaks dom nodes when closed. → [patch] [cla] TabContainer: closable tab pane leaks dom nodes when closed. |
Cool, leak fixes always appreciated! BTW if your attachments have a .patch or .diff extension they show up formatted nicely, see for example http://bugs.dojotoolkit.org/attachment/ticket/5417/5417.patch
comment:2 Changed 12 years ago by
Which version is your patch against? I tried it against the 1.1.1 StackContainer?.js [13089] and the latest [14420] but it match up.
comment:3 Changed 12 years ago by
| Status: | new → assigned |
|---|
comment:4 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
test demonstrating closable tab pane leak when run in sIEve