Closed Bug 579867 Opened 9 years ago Closed 9 years ago

Make foreign content end tag handling not loop back after processing the token in the secondary insertion mode

Categories

(Core :: HTML: Parser, defect, P2, major)

defect

Tracking

()

RESOLVED FIXED
mozilla2.0b4
Tracking Status
blocking2.0 --- betaN+

People

(Reporter: hsivonen, Assigned: hsivonen)

References

()

Details

(Keywords: compat)

Attachments

(1 file, 3 obsolete files)

Hixie changed the spec to fix some spec bugs. Need to implement those changes and remove previous workarounds for the spec bugs.
Blocks: 577416
Blocks: 577418
Priority: -- → P2
Blocks: 574884
The specific spec change is
http://html5.org/tools/web-apps-tracker?from=5154&to=5155

(There's also http://html5.org/tools/web-apps-tracker?from=5158&to=5159 but I believe that spec change introduces a spec bug.)
Summary: Implement July 2010 spec changes to SVG in HTML → Make foreign content end tag handling not loop back after processing the token in the secondary insertion mode
Requesting blocker status, because failure to fix this leads to all sorts of badness (the dependent bugs plus at least three bugs that already have temporary workarounds in place).
blocking2.0: --- → ?
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Attachment #458987 - Flags: review?(jonas)
blocking2.0: ? → betaN+
Keywords: compat
Attached patch The actual patch from hg export (obsolete) — Splinter Review
Attaching the actual patch (from hg export; without -w). This is the same change as attachment 458987 [details] [diff] [review]. Attachment 458987 [details] [diff] is more readable, but I'm moving the review request onto the actual patch that includes the white space changes and a check-in comment.
Attachment #462033 - Flags: review?(jonas)
Attachment #458987 - Flags: review?(jonas)
Comment on attachment 462033 [details] [diff] [review]
The actual patch from hg export

rs=me
Attachment #462033 - Flags: review?(jonas) → review+
I'm going on vacation today, and the tree is closed. Furthermore, it will be too late for beta 4 when I'm returning. Hence, checkin-needed.
Attachment #458987 - Attachment is obsolete: true
Attachment #458989 - Attachment is obsolete: true
Attachment #462033 - Attachment is obsolete: true
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/16bc32053260
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite+
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b4
Uh, parser/htmlparser/tests/reftest/bug566280-1-ref.html~ (note the ~) probably shouldn't have been checked in.
You need to log in before you can comment on or make changes to this bug.