Opened 13 years ago
Closed 13 years ago
#4047 closed defect (fixed)
IE6 slowdown, memory leak
Reported by: | bill | Owned by: | bill |
---|---|---|---|
Priority: | high | Milestone: | 0.9 |
Component: | Dijit | Version: | 0.9 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Every time you reload themeTester.html IE6's memory usage goes up 10M, and it takes 4s longer to parse the page then before.
Attachments (1)
Change History (8)
comment:1 Changed 13 years ago by
Component: | General → Dijit |
---|---|
Milestone: | → 0.9 |
Owner: | changed from anonymous to bill |
comment:2 Changed 13 years ago by
comment:3 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:5 Changed 13 years ago by
Attached a patch to remove dojo.forEach calls with inline functions. Function setup and tear down is expensive in tight loops. If we aren't gaining code reuse, dojo.disconnect for example, I would suggest removing inlined forEach calls.
schallm
Changed 13 years ago by
Attachment: | _Widget.patch added |
---|
comment:6 Changed 13 years ago by
comment:7 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Note: See
TracTickets for help on using
tickets.
Problem goes away if you remove the destroyRendering() call from Widget.destroy(), thus leaving the widgets in the document making it easier for IE to garbage collect.
But, the ability to dynamically create/destroy widgets (including their dom nodes) is important for long running apps like yahoo mail.
Will try tracking all dojoAttachPoints and deleting them on widget destruction. dojoAttachEvent calls are already being released...