Open
Bug 940282
Opened 11 years ago
Updated 3 years ago
View Source doesn't flag invalid character as error
Categories
(Core :: DOM: HTML Parser, defect, P5)
Core
DOM: HTML Parser
Tracking
()
UNCONFIRMED
People
(Reporter: sideshowbarker, Unassigned)
References
()
Details
view-source:http://w3c-test.org/web-platform-tests/master/conformance-checkers/html/parser/004-novalid.html The  character should be flagged as an error. Other test cases that exhibit the same problem are at: http://w3c-test.org/web-platform-tests/master/conformance-checkers/html/parser/ e.g., view-source:http://w3c-test.org/web-platform-tests/master/conformance-checkers/html/parser/001-novalid.html (there's a U+000B character in the body of the document)
Updated•11 years ago
|
Flags: needinfo?(hsivonen)
Comment 1•10 years ago
|
||
The basic problem here is that in Gecko, the code for reading a character is mere array access without checking the character for bogosity. I supposed we could look into using template magic to compile a different "read character" code path for View Source. But in that case, we'd then need a way to attach errors to individual characters. So this is intentional in the sense that I knowingly didn't take the time to implement all this, but in principle, it would be nice to have code for this.
Flags: needinfo?(hsivonen)
Reporter | ||
Comment 2•10 years ago
|
||
(In reply to Henri Sivonen (:hsivonen) from comment #1) > So this is intentional in the sense that I knowingly didn't take the time to > implement all this, but in principle, it would be nice to have code for this. OK, understood. I raised it in spirit with the same nice-to-have principle you mentioned, so I don't feel strongly about it (certainly not strongly enough to take time to write up a patch for it myself So, given all that, I think moving this to resolved=wontfix would be fine. I don't think it's worth keeping open if there's not a need/activity to refine the behavior any time soon to add it.
Comment 3•3 years ago
|
||
Bulk-downgrade of unassigned, >=5 years untouched DOM/Storage bugs' priority.
If you have reason to believe this is wrong (especially for the severity), please write a comment and ni :jstutte.
Severity: normal → S4
Priority: -- → P5
You need to log in
before you can comment on or make changes to this bug.
Description
•