Opened 15 years ago
Closed 10 years ago
#5774 closed Bug (wontfix)
[IE6] Memory Leak revisited
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | General | Version: | |
| Keywords: | IE | Cc: | kenny.demetter@… |
Description ¶
Sieve is still showing multiple DOM leaks on page left, it's pretty easy to reproduce with the default sample page, and this only affects IE6.
Find this issue on GitHubChange History (5)
comment:1 Changed 15 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | new → assigned |
Changed 15 years ago by
| Attachment: | 5774.patch added |
|---|
comment:2 Changed 15 years ago by
| Keywords: | Review? added |
|---|
The patch also introduces 'unload' event hooks described here that is required for this cleanup.
comment:3 Changed 15 years ago by
| Keywords: | IE added; Confirmed IE? removed |
|---|---|
| Milestone: | CKEditor 3.4 |
comment:4 Changed 10 years ago by
| Resolution: | → wontfix |
|---|---|
| Status: | review → closed |
IE6 is no longer aupported in CKE 4.x.
Find this issue on GitHubNote: See
TracTickets for help on using
tickets.
Based on my testing, it could hardly form a event handler cycle reference leak pattern, as we have well-established defense of this, and invalidate all the event listener doesn't reduce one single leak count.
Unfortunately I was not unable to figure out where exactly comes the leak but did find where to dereference to eliminate all the leaks - releasing our dom wrapper's '$' reference does the trick.