Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#13411 closed Bug (invalid)

"too much recursion" firefox error with 100+ CKE instances

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

Description

Bug demo: https://jsfiddle.net/umjyd4oj/3/

Demo creates 150 textarea and replaces them with CKEditors. This will result in "too much recursion" error in Firefox. And only first 95 textareas will be properly replaced. (this number varies?)

Demo fails in Firefox, but works without errors in Chromium Tested in Firefox 38.0.1 (64-bit) and 34.0 (64-bit) Works wihout erros on Chomium 43.0.2357.65 (64-bit)

Change History (1)

comment:1 Changed 4 years ago by Jakub Ś

Resolution: invalid
Status: newclosed
Version: 4.4.7

150 editors in a loop??? Editor is complicated application which creates its objects in browser's memory, attaches event handlers, creates new DOM elements etc.
Creating 150 instances at once might use all browser available memory.

If this is real life use case and you can't avoid it, please experiment with setTimeout e.g. https://jsfiddle.net/umjyd4oj/7/.[[BR]] I think however that better approach would be creating CKEditor on demand e.g. when user clicks into textarea. Also another approach is creating inline editors (instead of “iframed” ones) as shown here: https://jsfiddle.net/umjyd4oj/8/ (timeout value can be decreased).

I'm sorry but I can't accept this as a bug. @rwman if you don't agree, please explain your use case in detail.

Last edited 4 years ago by Jakub Ś (previous) (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy