Closed Bug 552908 Opened 10 years ago Closed 10 years ago

[HTML5] Implement spec changes to SVG </a> and </font> handling

Categories

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

defect

Tracking

()

RESOLVED FIXED
Tracking Status
blocking2.0 --- final+

People

(Reporter: hsivonen, Assigned: hsivonen)

References

Details

Attachments

(1 file)

It appears that Hixie hasn't gotten to the spec bugs yet. Since it's not quite nice to ship an alpha with this bug in it, I'm attaching my current WIP patch where I've implemented http://html5.org/tools/web-apps-tracker?from=4802&to=4803 but improvised around the three spec bugs that I found and mentioned in an earlier comment.

I suggest landing this for alpha and adjusting the code once Hixie has fixed the spec bugs.

Unfortunately, this patch changes indent (again) in a way that makes the patch annoying to review even though I tried to ask hg diff to ignore whitespace changes. It helps to focus on the start and end of the endTag method and assume that the stuff in the big |switch (mode)| stays as-is.
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Attachment #447320 - Flags: review?(jonas)
blocking2.0: --- → ?
Blocking.
blocking2.0: ? → final+
Comment on attachment 447320 [details] [diff] [review]
Implement spec rev 4803 improvising around the bugs in the spec rev

rs=me

Would be great if you could try to break up these patches into indenting-changing parts and functional-changing parts or some such. The patch as it is can't really be reviewed :(
Attachment #447320 - Flags: review?(jonas) → review+
Pushed: http://hg.mozilla.org/mozilla-central/rev/62c09fac599a
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Depends on: 577416
You need to log in before you can comment on or make changes to this bug.