1
edit
Changes
→Bugs To Be Filed
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:
Files that this [https://bug629350.bugzilla.mozilla.org/attachment.cgi?id=704716 initial patch] touches:
toolkit/toolkit-tiers.mk
</pre>
==Bugs To Be Filed==
1. nsVideoFrame changes
* Assigned To: Jesse Silver
* Description: "[webvtt] Add captions div to nsVideoFrame for webvtt subtitle display"
* Product: Core
* Component: Layout
* Bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=833388
* 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: caitp
* Description: "[webvtt] Integrate libwebvtt parser into build system"
* Product: Core
* Component: Video/Audio
* Bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=833403
* 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: Rick Eyre, Shayan Zafar
* Description: "[webvtt] Implement TextTrackDecoder to manage the libwebvtt decoder"
* Product: Core
* Component: Video/Audio
* Bug: https://bugzilla.mozilla.org/show_bug.cgi?id=833382
* 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: Dale
* Description: "[webvtt] Implement Track element and TextTrack* DOM classes"
* Product: Core
* Component: Video/Audio
* Bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=833385
* 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: jbraffoul, msaad
* Description: "[webvtt] Tests for Track element and TextTrack* DOM classes"
* Product: Core
* Component: Video/Audio
* Bugzilla Link : https://bugzilla.mozilla.org/show_bug.cgi?id=833386