Opened 3 years ago

Last modified 3 years ago

#11906 confirmed Bug

Parser does not accept incorrect HTML tag with excessive quotes

Reported by: wenbinke Owned by:
Priority: Normal Milestone:
Component: Core : Parser Version:
Keywords: Cc: wenbinke@…

Description (last modified by Reinmar)

See comment:2 for TC. Note that this HTML is not fixed by Firefox nor Chrome when we pass content through fixBin.

Change History (3)

comment:1 Changed 3 years ago by Reinmar

First of all, the HTML is invalid. There's an excessive quote after frameborder.

This was causing editor crash on Chrome and this issue is already fixed on master (#11754).

Regarding the encoding - that's also caused by the invalid HTML. We'll discuss whether we want to fix editor's parser or whether we'll mark this as "won't fix", because that's not a bug in CKEditor.

comment:2 Changed 3 years ago by Reinmar

The TC to test parser:

var frag = CKEDITOR.htmlParser.fragment.fromHtml( '<iframe src="x" frameborder="0" " width=" 560"></iframe></p>' );

// Expected - element.
// Actual - text.
frag.children[ 0 ];

comment:3 Changed 3 years ago by Reinmar

  • Component changed from General to Core : Parser
  • Description modified (diff)
  • Status changed from new to confirmed
  • Summary changed from Embeded youtube iframe code makes chrome stuck to Parser does not accept incorrect HTML tag with excessive quotes
  • Version 4.4.0 deleted

We confirm this issue, but we're not planning to work on it (at least not now). Issue is caused by creating incorrect HTML in source mode and CKEditor does not have to accept every broken thing.

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