Opened 9 years ago
Last modified 9 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 )
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 9 years ago by
comment:2 Changed 9 years ago by
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 9 years ago by
Component: | General → Core : Parser |
---|---|
Description: | modified (diff) |
Status: | new → confirmed |
Summary: | Embeded youtube iframe code makes chrome stuck → Parser 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.
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.