Opened 7 years ago

Closed 7 years 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 7 years ago by Jakub Ś

Keywords: Support added
Status: newconfirmed

comment:2 Changed 7 years ago by Marek Lewandowski

cc

comment:3 Changed 7 years ago by Marek Lewandowski

Description: modified (diff)

comment:4 Changed 7 years ago by Marek Lewandowski

Owner: set to Marek Lewandowski
Status: confirmedreview

Pushed the fix to branch:t/16600.

Last edited 7 years ago by Marek Lewandowski (previous) (diff)

comment:5 Changed 7 years ago by kkrzton

Status: reviewreview_passed

LGTM!

comment:6 Changed 7 years ago by kkrzton

Resolution: fixed
Status: review_passedclosed

Merged to master with a523444.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy