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

Status: RESOLVED FIXED

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

Description

[Henri Sivonen (:hsivonen)]

Implement
http://www.w3.org/Bugs/Public/show_bug.cgi?id=8767
http://html5.org/tools/web-apps-tracker?from=4802&to=4803

Comment 1

[Henri Sivonen (:hsivonen)]

Blocking on
http://www.w3.org/Bugs/Public/show_bug.cgi?id=9580
http://www.w3.org/Bugs/Public/show_bug.cgi?id=9581
http://www.w3.org/Bugs/Public/show_bug.cgi?id=9582

Comment 2

[Henri Sivonen (:hsivonen)]

Attachment: Implement spec rev 4803 improvising around the bugs in the spec rev

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.

Comment 3

[Johnny Stenback (:jst)]

Blocking.

Comment 4

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

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 :(

Comment 5

[Henri Sivonen (:hsivonen)]

Pushed: http://hg.mozilla.org/mozilla-central/rev/62c09fac599a