Open
Bug 1259039
Opened 10 years ago
Updated 3 years ago
<option>EOF and <optgroup>EOF should not be parse errors
Categories
(Core :: DOM: HTML Parser, defect)
Core
DOM: HTML Parser
Tracking
()
ASSIGNED
| Tracking | Status | |
|---|---|---|
| firefox48 | --- | affected |
People
(Reporter: zcorpan, Assigned: hsivonen)
Details
(Whiteboard: btpp-active)
Attachments
(1 file)
|
1.58 KB,
patch
|
wchen
:
review+
|
Details | Diff | Splinter Review |
https://parsetree.validator.nu/?parser=html5&content=<%21DOCTYPE+html><option>&submit=Print+Tree
https://parsetree.validator.nu/?parser=html5&content=<%21DOCTYPE+html><optgroup>&submit=Print+Tree
Should have no errors.
[[
If there is a node in the stack of open elements that is not either a dd element, a dt element, an li element, an optgroup element, an option element, a p element, an rb element, an rp element, an rt element, an rtc element, a tbody element, a td element, a tfoot element, a th element, a thead element, a tr element, the body element, or the html element, then this is a parse error.
]]
https://html.spec.whatwg.org/multipage/syntax.html#parsing-main-inbody
Similar for <option></html> (it seems <option></body> does not get a parse error with the V.nu HTML parser).
Comment 1•10 years ago
|
||
Henri, how soon do you think you can get to this?
Flags: needinfo?(hsivonen)
Whiteboard: btpp-followup-2016-04-26
| Assignee | ||
Comment 2•10 years ago
|
||
Hopefully next week, but let's say the week after.
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Flags: needinfo?(hsivonen)
Comment 3•10 years ago
|
||
I was thinking in terms of months so that's great :)
Whiteboard: btpp-followup-2016-04-26 → btpp-active
| Assignee | ||
Comment 4•10 years ago
|
||
This is not actually observable in Firefox, but this should still land on m-c to keep the sources in sync. Test cases should go into html5lib.
Attachment #8748756 -
Flags: review?(wchen)
| Assignee | ||
Comment 5•10 years ago
|
||
Updated•10 years ago
|
Attachment #8748756 -
Flags: review?(wchen) → review+
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•