Closed
Bug 287349
Opened 19 years ago
Closed 19 years ago
<select> with an unknown tag inside is broken
Categories
(Core :: DOM: HTML Parser, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: emaijala+moz, Assigned: mrbkap)
References
Details
(Keywords: regression, testcase)
Attachments
(2 files)
252 bytes,
text/html
|
Details | |
1.09 KB,
patch
|
bzbarsky
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
I'm not sure if the component is correct. This is a regression between nightly trunk builds 20050317 and 20050318 at least on WinXP. An unknown tag inside a <select> tag now seems to close the <select> tag. The following <option> tags are dumped on the screen. This is admittedly bad markup, but build up to 20050317 were able to cope with it. I'll add a test case.
Reporter | ||
Comment 1•19 years ago
|
||
This is a test case reduced from a real-world problem case.
Assignee | ||
Comment 2•19 years ago
|
||
This is my regression from bug 285250. userdefined tags used to be solely kFlowEntitys, so we excluded them (and later dropped them). Now they're both kFlowEntity AND kHeadMisc tags, so we don't exclude them, and they close out the select. Potential patch coming in a moment.
Assignee: parser → mrbkap
Assignee | ||
Comment 3•19 years ago
|
||
This patch restores the old behavior by allowing <select> to exclude userdefined tags.
Attachment #178350 -
Flags: superreview?(jst)
Attachment #178350 -
Flags: review?(bzbarsky)
Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Comment 4•19 years ago
|
||
Comment on attachment 178350 [details] [diff] [review] patch v1 r=bzbarsky
Attachment #178350 -
Flags: review?(bzbarsky) → review+
Comment 5•19 years ago
|
||
Comment on attachment 178350 [details] [diff] [review] patch v1 sr=jst
Attachment #178350 -
Flags: superreview?(jst) → superreview+
Assignee | ||
Comment 6•19 years ago
|
||
Fix checked in. Sorry for the inconvenience.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Comment 7•19 years ago
|
||
*** Bug 287606 has been marked as a duplicate of this bug. ***
Updated•19 years ago
|
Flags: blocking1.8b2?
You need to log in
before you can comment on or make changes to this bug.
Description
•