Closed
Bug 262346
Opened 20 years ago
Closed 20 years ago
Missing > followed by a < causes us to not apply the first tag
Categories
(Core :: DOM: HTML Parser, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mrbkap, Assigned: mrbkap)
Details
(Keywords: regression)
Attachments
(2 files)
146 bytes,
text/html
|
Details | |
976 bytes,
patch
|
bzbarsky
:
review+
bzbarsky
:
superreview+
|
Details | Diff | Splinter Review |
I'll attach a testcase in a second. This was broken by the checkin for bug
258082. I have a fix (which I'll also attach).
Assignee | ||
Comment 1•20 years ago
|
||
Self-explanatory testcase, view using the trunk (i.e., not avairy).
Assignee | ||
Comment 2•20 years ago
|
||
Bad attributes don't mean that we should immediately close the current tag.
Assignee | ||
Comment 3•20 years ago
|
||
Comment on attachment 160675 [details] [diff] [review]
patch v1
One note: I think the comment I added should be in the |if| portion of that
branch, but I didn't want to spam the bug with another patch.
Attachment #160675 -
Flags: superreview?(bzbarsky)
Attachment #160675 -
Flags: review?(bzbarsky)
![]() |
||
Comment 4•20 years ago
|
||
Comment on attachment 160675 [details] [diff] [review]
patch v1
r+sr=bzbarsky
Attachment #160675 -
Flags: superreview?(bzbarsky)
Attachment #160675 -
Flags: superreview+
Attachment #160675 -
Flags: review?(bzbarsky)
Attachment #160675 -
Flags: review+
Assignee | ||
Comment 5•20 years ago
|
||
This was checked in. Marking FIXED.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•