[CKEditor Dev] #11010: Fix memory leaks caused by CKEDITOR.filter.instances storing nested editables' filters forever

107 views
Skip to first unread message

CKEditor

unread,
Oct 18, 2013, 8:17:34 AM10/18/13
to ckeditor...@googlegroups.com
#11010: Fix memory leaks caused by CKEDITOR.filter.instances storing nested
editables' filters forever
--------------------------+----------------------------------
Reporter: Reinmar | Owner:
Type: Bug | Status: new
Priority: Normal | Milestone: CKEditor 4.3.1
Component: UI : Widgets | Version: 4.3 (GitHub - major)
Keywords: |
--------------------------+----------------------------------
Every nested editable which has allowed content rules defined has its
filter instance. These instances are stored in nested editable instances,
which most likely are correctly garbage collected. But filter instances
are also stored in CKEDITOR.filter.instances and they are kept there
forever.

We could have filter.destroy() method removing instance from the instances
hash and doing other cleanup if necessary. Then we could destroy filters
when destroying widgets.

--
Ticket URL: <http://dev.ckeditor.com/ticket/11010>
CKEditor <http://ckeditor.com/>
The text editor for the Internet

CKEditor

unread,
Oct 21, 2013, 9:51:11 AM10/21/13
to ckeditor...@googlegroups.com
#11010: Fix memory leaks caused by CKEDITOR.filter.instances storing nested
editables' filters forever
---------------------------+----------------------------------
Reporter: Reinmar | Owner:
Type: Bug | Status: confirmed

Priority: Normal | Milestone: CKEditor 4.3.1
Component: UI : Widgets | Version: 4.3 (GitHub - major)
Resolution: | Keywords:
---------------------------+----------------------------------
Changes (by j.swiderski):

* status: new => confirmed


--
Ticket URL: <http://dev.ckeditor.com/ticket/11010#comment:1>

CKEditor

unread,
Nov 18, 2013, 9:20:20 AM11/18/13
to ckeditor...@googlegroups.com
#11010: Fix memory leaks caused by CKEDITOR.filter.instances storing nested
editables' filters forever
---------------------------+----------------------------
Reporter: Reinmar | Owner:
Type: Bug | Status: confirmed
Priority: Normal | Milestone: CKEditor 4.3.2

Component: UI : Widgets | Version: 4.3
Resolution: | Keywords:
---------------------------+----------------------------
Changes (by Reinmar):

* milestone: CKEditor 4.3.1 => CKEditor 4.3.2


--
Ticket URL: <http://dev.ckeditor.com/ticket/11010#comment:2>

CKEditor

unread,
Dec 8, 2013, 4:31:38 PM12/8/13
to ckeditor...@googlegroups.com
#11010: Fix memory leaks caused by CKEDITOR.filter.instances storing nested
editables' filters forever
---------------------------+--------------------------
Reporter: Reinmar | Owner:
Type: Bug | Status: confirmed
Priority: Normal | Milestone: CKEditor 4.4

Component: UI : Widgets | Version: 4.3
Resolution: | Keywords:
---------------------------+--------------------------
Changes (by Reinmar):

* milestone: CKEditor 4.3.2 => CKEditor 4.4


Comment:

This requires new features to be implemented, so 4.4 is more reasonable.

--
Ticket URL: <http://dev.ckeditor.com/ticket/11010#comment:3>

CKEditor

unread,
Feb 4, 2014, 6:16:17 AM2/4/14
to ckeditor...@googlegroups.com
#11010: Fix memory leaks caused by CKEDITOR.filter.instances storing nested
editables' filters forever
---------------------------+-----------------------
Reporter: Reinmar | Owner:
Type: Bug | Status: confirmed
Priority: Normal | Milestone:

Component: UI : Widgets | Version: 4.3
Resolution: | Keywords:
---------------------------+-----------------------
Changes (by Reinmar):

* milestone: CKEditor 4.4 =>


Comment:

Unfortunately, we'll not have enough time in 4.4 to handle this.

--
Ticket URL: <http://dev.ckeditor.com/ticket/11010#comment:4>

CKEditor

unread,
Dec 11, 2015, 3:30:15 AM12/11/15
to ckeditor...@googlegroups.com
#11010: Fix memory leaks caused by CKEDITOR.filter.instances storing nested
editables' filters forever
---------------------------+-----------------------
Reporter: Reinmar | Owner:
Type: Bug | Status: confirmed
Priority: Normal | Milestone:

Component: UI : Widgets | Version: 4.3
Resolution: | Keywords:
---------------------------+-----------------------

Comment (by seadmemic):

Will there be any progress soon on this bug?

--
Ticket URL: <http://dev.ckeditor.com/ticket/11010#comment:5>

CKEditor

unread,
Dec 15, 2015, 5:33:59 AM12/15/15
to ckeditor...@googlegroups.com
#11010: Fix memory leaks caused by CKEDITOR.filter.instances storing nested
editables' filters forever
---------------------------+-----------------------
Reporter: Reinmar | Owner:
Type: Bug | Status: confirmed
Priority: Normal | Milestone:

Component: UI : Widgets | Version: 4.3
Resolution: | Keywords:
---------------------------+-----------------------

Comment (by j.swiderski):

No one is currently working on this ticket right now, but thank you for
team reminder.

I have talked with the team and we will consider giving this ticket a try
in one of future releases (by future I mean no sooner than 4.5.8).

NOTE: You are welcome to leave another reminder one CKEditor 4.5.7 is out.

--
Ticket URL: <http://dev.ckeditor.com/ticket/11010#comment:6>

CKEditor

unread,
May 22, 2017, 8:57:25 AM5/22/17
to ckeditor...@googlegroups.com
#11010: Fix memory leaks caused by CKEDITOR.filter.instances storing nested
editables' filters forever
---------------------------+-----------------------
Reporter: Reinmar | Owner:
Type: Bug | Status: confirmed
Priority: Normal | Milestone:

Component: UI : Widgets | Version: 4.3
Resolution: | Keywords:
---------------------------+-----------------------

Comment (by ehudm1):

Any news about this defect?

--
Ticket URL: <http://dev.ckeditor.com/ticket/11010#comment:7>

CKEditor

unread,
Jun 30, 2017, 5:45:04 AM6/30/17
to ckeditor...@googlegroups.com
#11010: Fix memory leaks caused by CKEDITOR.filter.instances storing nested
editables' filters forever
---------------------------+-----------------------
Reporter: Reinmar | Owner:
Type: Bug | Status: confirmed
Priority: Normal | Milestone:

Component: UI : Widgets | Version: 4.3
Resolution: | Keywords:
---------------------------+-----------------------

Comment (by j.swiderski):

Other related issues: #16869, #12307, #11010, #13123.

--
Ticket URL: <http://dev.ckeditor.com/ticket/11010#comment:8>

Reply all
Reply to author
Forward
0 new messages