Opened 9 years ago
Closed 8 years ago
#14618 closed Bug (expired)
Memory leak happening in CKEditor when navigating to different view
Reported by: | Kunal Garg | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 4.5.9 |
Keywords: | memory-leak | Cc: |
Description
Steps to reproduce
- Use CKEditor.replace on textArea
- Now navigate to different view. This destroy the editor.
- Recreate the editor with CKEditor.replace
Expected result
The objects allocated in the first iteration should be fred
Actual result
The objects are still there and keep on piling on repeating step 2 and 3
Other details (browser, OS, CKEditor version, installed plugins)
Attachments (1)
Change History (5)
Changed 9 years ago by
Attachment: | Capture.PNG added |
---|
comment:1 Changed 9 years ago by
I debugged the issue and it is happening because of following code: form.$.submit = CKEDITOR.tools.override( form.$.submit, function( originalSubmit ) {
return function() {
onSubmit();
For IE, the DOM submit function is not a function, so we need third check. if ( originalSubmit.apply )
originalSubmit.apply( this );
else
originalSubmit();
};
});
It works fine after I set the form.$.submit = null; in destroy.
comment:2 Changed 9 years ago by
The attachment shows hw objects are retained by the submit of form element
comment:3 Changed 9 years ago by
Status: | new → pending |
---|
I'm having trouble reproducing this one. Could you provide a heap snapshot that demonstrates this issue?
comment:4 Changed 8 years ago by
Resolution: | → expired |
---|---|
Status: | pending → closed |
Showing memory objects bound to the FormElement