Opened 14 years ago

Closed 9 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@…


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.

Attachments (1)

5774.patch (5.1 KB) - added by Garry Yao 14 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 14 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

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.

Changed 14 years ago by Garry Yao

Attachment: 5774.patch added

comment:2 Changed 14 years ago by Garry Yao

Keywords: Review? added

The patch also introduces 'unload' event hooks described here that is required for this cleanup.

comment:3 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: IE added; Confirmed IE? removed
Milestone: CKEditor 3.4

comment:4 Changed 9 years ago by Jakub Ś

Resolution: wontfix
Status: reviewclosed

IE6 is no longer aupported in CKE 4.x.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy