Opened 11 years ago
Last modified 11 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 11 years ago by
comment:2 Changed 11 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 11 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.