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 Piotrek Koszuliński)

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 Piotrek Koszuliński

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 Piotrek Koszuliński

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 Piotrek Koszuliński

Component: GeneralCore : Parser
Description: modified (diff)
Status: newconfirmed
Summary: Embeded youtube iframe code makes chrome stuckParser does not accept incorrect HTML tag with excessive quotes
Version: 4.4.0

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 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy