Opened 10 months ago

Closed 9 months ago

#16600 closed Bug (fixed)

Editor may be assuming presence of editable too optimistically.

Reported by: Jakub Ś Owned by: Marek Lewandowski
Priority: Normal Milestone: CKEditor 4.6.1
Component: General Version:
Keywords: Webkit Blink Support Cc:

Description (last modified by Marek Lewandowski)

Steps to reproduce

Sometimes in Blink and Webkit browsers when dealing with multiple editors you may encounter a situation where editable is not present yet.

URI: core/dom/element.js
Line: 1986

Code: if ( editable.equals( this )
editable.contains( this ) ) {

Expected result

No error should be thrown.

Actual result

In these rare cases error might get thrown.

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

Problem occurs in Blink and Webkit browsers and simple null check should do the trick.

Change History (6)

comment:1 Changed 10 months ago by Jakub Ś

Keywords: Support added
Status: newconfirmed

comment:2 Changed 10 months ago by Marek Lewandowski

cc

comment:3 Changed 9 months ago by Marek Lewandowski

Description: modified (diff)

comment:4 Changed 9 months ago by Marek Lewandowski

Owner: set to Marek Lewandowski
Status: confirmedreview

Pushed the fix to branch:t/16600.

Last edited 9 months ago by Marek Lewandowski (previous) (diff)

comment:5 Changed 9 months ago by kkrzton

Status: reviewreview_passed

LGTM!

comment:6 Changed 9 months ago by kkrzton

Resolution: fixed
Status: review_passedclosed

Merged to master with a523444.

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