Opened 3 years ago

#13902 new Bug

SetData gets slower and slower

Reported by: acy Owned by:
Priority: Normal Milestone:
Component: General Version: 4.5.3
Keywords: Cc:

Description

Steps to reproduce

I'm sorry as i was not able to reproduce this in a simple case, but i found useful to report this bug anyway since I found at least a part of the solution. We use multiple CKEditors in a react application and we only have this bug in a particular and complex situation. Each time we try to simplify the context, the bug disappears.

Actual result

Each time we call SetData, it gets 2 times slower. This may be related to this bug : http://stackoverflow.com/questions/21388256/ckeditor-memory-leak-on-setdata

The problem comes from the clearListeners function of the editable.js file. For an unknown reason, some listeners become undefined, and we get out of the function before all listeners are removed, causing "afterSetData" listeners number to be doubled each call. Simply moving the try/catch inside the while solves the problem, but doesn't explain why some listeners are undefined. Hope this helps.

Acy

Other details (browser, OS, CKEditor version, installed plugins)

Windows 7 64bit, Chrome, CKEditor 4.5.3

Change History (0)

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy