The default bug view has changed. See this FAQ.

Add support for the <track> element to the HTML parser

RESOLVED FIXED

Status

()

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

People

(Reporter: hsivonen, Assigned: hsivonen)

Tracking

(Blocks: 1 bug, {dev-doc-complete})

unspecified
dev-doc-complete
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Assignee)

Description

6 years ago
The HTML5 spec now requires <track> to parse like <param> and <source>. Mike(tm) Smith has patched the Java version of the parser. Need to land the translation into m-c.
(Assignee)

Comment 1

6 years ago
Created attachment 499005 [details] [diff] [review]
Add track element support to the parser

Mike, are you ok with being the hg author of record for the code that is autogenerated from your Java patch?
Attachment #499005 - Flags: review+
Attachment #499005 - Flags: approval2.0?
(Assignee)

Comment 2

6 years ago
Created attachment 499006 [details] [diff] [review]
Add tests for void elements
Attachment #499006 - Flags: review?(jgriffin)
Attachment #499006 - Flags: review?(jgriffin) → review+

Comment 3

6 years ago
(In reply to comment #1)
> Mike, are you ok with being the hg author of record for the code that is
> autogenerated from your Java patch?

Yes

Updated

6 years ago
Attachment #499005 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 4

6 years ago
Thanks. Landed.

http://hg.mozilla.org/mozilla-central/rev/912a9991c5c6
http://hg.mozilla.org/mozilla-central/rev/9497c1308688
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED

Comment 5

6 years ago
Don't forget "dev-doc-needed" keyword for those bugs, otherwise documentation is a game of hazard.
Keywords: dev-doc-needed
Adding link to specification.
Documentation updated to indicate Firefox compatibility:

https://developer.mozilla.org/en/HTML/Element/track

Also listed on Firefox 6 for developers.
Keywords: dev-doc-needed → dev-doc-complete
If I understand well this bug, the documentation is not accurate. Support for <track> wasn't the goal of this bug, merely its support by the HTML parser.

In other terms, the compatibility matrix in https://developer.mozilla.org/en/HTML/Element/track shouldn't have been updated (I've corrected it).

And in Firefox 6 for developers, I've updated the sentence to be more accurate.

(I got feedback from a web developer trying to use this feature using Aurora and the MDN information — yes! There are some people using these! — and surprised that it failed)
Blocks: 629350
(Assignee)

Comment 9

6 years ago
(In reply to comment #8)
> If I understand well this bug, the documentation is not accurate. Support
> for <track> wasn't the goal of this bug, merely its support by the HTML
> parser.

Well, this bug is in the parser component. Resummarizing to make things clearer.
Summary: Add support for the <track> element → Add support for the <track> element to the HTML parser
You need to log in before you can comment on or make changes to this bug.