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

RESOLVED FIXED

Status

()

Core
HTML: Parser
P2
normal
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: hsivonen, Assigned: hsivonen)

Tracking

unspecified
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(blocking2.0 final+)

Details

Attachments

(1 attachment)

(Assignee)

Description

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

Comment 1

8 years ago
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
(Assignee)

Comment 2

8 years ago
Created attachment 447320 [details] [diff] [review]
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.
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Attachment #447320 - Flags: review?(jonas)
(Assignee)

Updated

8 years ago
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+
(Assignee)

Comment 5

8 years ago
Pushed: http://hg.mozilla.org/mozilla-central/rev/62c09fac599a
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Depends on: 568611

Updated

8 years ago
Depends on: 577416
You need to log in before you can comment on or make changes to this bug.