Opened 9 years ago

Closed 4 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.

Attachments (1)

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

Download all attachments as: .zip

Change History (5)

comment:1 Changed 9 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 9 years ago by Garry Yao

Attachment: 5774.patch added

comment:2 Changed 9 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 9 years ago by Frederico Caldeira Knabben

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

comment:4 Changed 4 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 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy