Last Comment Bug 620664 - Add support for the <track> element to the HTML parser
: Add support for the <track> element to the HTML parser
Status: RESOLVED FIXED
: dev-doc-complete
Product: Core
Classification: Components
Component: HTML: Parser (show other bugs)
: unspecified
: All All
: -- normal with 1 vote (vote)
: ---
Assigned To: Henri Sivonen (:hsivonen)
:
Mentors:
http://dev.w3.org/html5/spec/Overview...
Depends on:
Blocks: webvtt
  Show dependency treegraph
 
Reported: 2010-12-21 04:47 PST by Henri Sivonen (:hsivonen)
Modified: 2011-07-04 16:40 PDT (History)
6 users (show)
hsivonen: in‑testsuite+
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Add track element support to the parser (49.34 KB, patch)
2010-12-21 05:56 PST, Henri Sivonen (:hsivonen)
hsivonen: review+
jst: approval2.0+
Details | Diff | Review
Add tests for void elements (6.51 KB, patch)
2010-12-21 05:56 PST, Henri Sivonen (:hsivonen)
jgriffin: review+
Details | Diff | Review

Description Henri Sivonen (:hsivonen) 2010-12-21 04:47:07 PST
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.
Comment 1 Henri Sivonen (:hsivonen) 2010-12-21 05:56:13 PST
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?
Comment 2 Henri Sivonen (:hsivonen) 2010-12-21 05:56:37 PST
Created attachment 499006 [details] [diff] [review]
Add tests for void elements
Comment 3 Michael[tm] Smith 2010-12-21 18:10:35 PST
(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
Comment 5 j.j. 2011-05-12 16:27:20 PDT
Don't forget "dev-doc-needed" keyword for those bugs, otherwise documentation is a game of hazard.
Comment 6 Eric Shepherd [:sheppy] 2011-05-16 15:25:53 PDT
Adding link to specification.
Comment 7 Eric Shepherd [:sheppy] 2011-05-23 14:59:09 PDT
Documentation updated to indicate Firefox compatibility:

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

Also listed on Firefox 6 for developers.
Comment 8 Jean-Yves Perrier [:teoli] 2011-06-09 00:28:02 PDT
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)
Comment 9 Henri Sivonen (:hsivonen) 2011-06-09 09:26:39 PDT
(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.

Note You need to log in before you can comment on or make changes to this bug.