667
edits
Changes
→Overview
We need to file smaller, secondary bugs to deal with the different parts of this work. That way we can (hopefully) land things in batches, and not block the whole thing on being 100% completed. Also, people can work in parallel.
==Major Implementation Tasks==
Ralph suggested an initial strategy, which I've expanded somewhat:
# the nsVideoFrame changes to add the display div. Flowing from this will be other layout considerations, like text positioning, CSS, and other display aspects of the WebVTT spec (not all of which we'll finish this term).
# the import and build support for the parser library in media/webvtt. This will include massaging libwebvtt's public API to work well with Mozilla.
# the WebVTTDecoder TextTrackDecoder (see [https://bugzilla.mozilla.org/show_bug.cgi?id=629350#c29 discussion]) implementation in content/media/webvtt
# DOM bindings for HTMLTrackElement, TextTrack, TextTrackCue, TextTrackCueList. Flowing from this will be the DOM API implementation for everything these objects can do.
# DOM Tests to test the new types (HTMLTrackElement, TextTrack, TextTrackCue, TextTrackCueList) and also any aspects of the WebVTT spec related to rendering.
toolkit/toolkit-tiers.mk
</pre>
==Bugs To Be Filed==
1. nsVideoFrame changes
* Assigned To: ?
* Description: "[webvtt] Add captions div to nsVideoFrame for webvtt subtitle display"
* Product: Core
* Component: Layout
* Files to consider from patch:
<pre>
layout/build/Makefile.in
layout/generic/nsVideoFrame.cpp
layout/generic/nsVideoFrame.h
layout/style/html.css
</pre>
2. Build support for libwebvtt parser
* Assigned To: ?
* Description: "[webvtt] Integrate libwebvtt parser into build system"
* Product: Core
* Component: Video/Audio
* Files to consider from patch:
<pre>
configure.in
media/webvtt/Makefile.in
media/webvtt/update.sh
layout/build/Makefile.in
toolkit/toolkit-tiers.mk
</pre>
3. TextTrackDecoder
* Assigned To: ?
* Description: "[webvtt] Implement TextTrackDecoder to manage the libwebvtt decoder"
* Product: Core
* Component: Video/Audio
* Files to consider from patch:
<pre>
content/media/Makefile.in
content/media/MediaDecoder.h
content/media/webvtt/Makefile.in
content/media/webvtt/nsWebVTTDecoder.cpp
content/media/webvtt/nsWebVTTDecoder.h
content/media/webvtt/nsWebVTTReader.cpp
content/media/webvtt/nsWebVTTReader.h
content/html/content/public/nsHTMLMediaElement.h
content/html/content/src/nsHTMLMediaElement.cpp
</pre>
4. DOM Bindings
* Assigned To: ?
* Description: "[webvtt] Implement Track element and TextTrack* DOM classes"
* Product: Core
* Component: Video/Audio
* Files to consider from patch:
<pre>
content/base/src/nsGkAtomList.h
content/html/content/src/HTMLTrackElement.cpp
content/html/content/src/HTMLTrackElement.h
content/html/content/src/Makefile.in
content/html/content/src/nsGenericHTMLElement.h
dom/media/Makefile.in
dom/media/TextTrack.cpp
dom/media/TextTrack.h
dom/media/TextTrackCue.cpp
dom/media/TextTrackCue.h
dom/media/TextTrackCueList.cpp
dom/media/TextTrackCueList.h
dom/webidl/HTMLTrackElement.webidl
dom/webidl/TextTrack.webidl
dom/webidl/TextTrackCue.webidl
dom/webidl/TextTrackCueList.webidl
dom/webidl/WebIDL.mk
editor/libeditor/html/nsHTMLEditUtils.cpp
parser/htmlparser/public/nsHTMLTagList.h
parser/htmlparser/src/nsElementTable.cpp
parser/htmlparser/src/nsHTMLTags.cpp
</pre>
5. DOM Tests
* Assigned To: ?
* Description: "[webvtt] Tests for Track element and TextTrack* DOM classes"
* Product: Core
* Component: Video/Audio
