Missing > followed by a < causes us to not apply the first tag

RESOLVED FIXED

Status

()

Core
HTML: Parser
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: mrbkap, Assigned: mrbkap)

Tracking

({regression})

Trunk
x86
Windows XP
regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

14 years ago
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

14 years ago
Created attachment 160674 [details]
testcase

Self-explanatory testcase, view using the trunk (i.e., not avairy).
(Assignee)

Comment 2

14 years ago
Created attachment 160675 [details] [diff] [review]
patch v1

Bad attributes don't mean that we should immediately close the current tag.
(Assignee)

Comment 3

14 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 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

14 years ago
This was checked in. Marking FIXED.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.