Opened 8 months ago

Closed 7 months ago

#16600 closed Bug (fixed)

Editor may be assuming presence of editable too optimistically.

Reported by: j.swiderski Owned by: m.lewandowski
Priority: Normal Milestone: CKEditor 4.6.1
Component: General Version:
Keywords: Webkit Blink Support Cc:

Description (last modified by m.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 8 months ago by j.swiderski

  • Keywords Support added
  • Status changed from new to confirmed

comment:2 Changed 8 months ago by m.lewandowski

cc

comment:3 Changed 7 months ago by m.lewandowski

  • Description modified (diff)

comment:4 Changed 7 months ago by m.lewandowski

  • Owner set to m.lewandowski
  • Status changed from confirmed to review

Pushed the fix to branch:t/16600.

Last edited 7 months ago by m.lewandowski (previous) (diff)

comment:5 Changed 7 months ago by k.krzton

  • Status changed from review to review_passed

LGTM!

comment:6 Changed 7 months ago by k.krzton

  • Resolution set to fixed
  • Status changed from review_passed to closed

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