Closed
Bug 882713
Opened 11 years ago
Closed 11 years ago
[webvtt] Implement TextTrackCue::active
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: reyre, Assigned: reyre)
References
()
Details
Attachments
(1 file)
5.16 KB,
patch
|
Details | Diff | Splinter Review |
This is a flag that's used in the time marches on algorithm.
Assignee | ||
Updated•11 years ago
|
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → rick.eyre
QA Contact: rick.eyre
Assignee | ||
Comment 1•11 years ago
|
||
This introduces support for the TextTrackCue::Active flag. One place that isn't handled that is in the spec is when it says to unset the active flag of all the cues in a text track if that text track is removed from a media element, but no where do we have an API to remove a text track from a media element that I can see.
Attachment #762755 -
Flags: review?(giles)
Assignee | ||
Comment 2•11 years ago
|
||
We've agreed for now that we won't be using TextTrackCue::active to be keeping track of the active text tracks. Instead we'll just be using cues whose start time and end time are valid for the current play back time as active.
I'm leaving this bug open for now in case we reconsider in the near future.
Assignee | ||
Updated•11 years ago
|
Attachment #762755 -
Flags: review?(giles)
Assignee | ||
Comment 3•11 years ago
|
||
I don't think we will be reconsidering in the near future. Using a sorted cue list and tracking the active cues via their start and end times is much more efficient for Gecko's solution.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•