Opened 10 years ago

Last modified 10 years ago

#11906 confirmed Bug

Parser does not accept incorrect HTML tag with excessive quotes — at Version 3

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 10 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 10 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 10 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 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy