Closed Bug 1279865 Opened 8 years ago Closed 8 years ago

[webvtt] fix html/semantics/embedded-content/media-elements/interfaces/TextTrack/activeCues.html

Categories

(Core :: Audio/Video: Playback, defect, P2)

Product:
Core
Component:
Audio/Video: Playback
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla50
Tracking Flags:
Tracking Status
firefox50 --- fixed

People

(Reporter: bechen, Assigned: bechen)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Bug 1279865 - Enable html/semantics/embedded-content/media-elements/interfaces/TextTrack/activeCues.html.
58 bytes, text/x-review-board-request
rillian
: review+
Details
Bug 1279865 - Don't run TimeMarchesOn if the MediaElement is not played.
58 bytes, text/x-review-board-request
rillian
: review+
Details
No description provided.
Comment on attachment 8762799 [details] Bug 1279865 - Enable html/semantics/embedded-content/media-elements/interfaces/TextTrack/activeCues.html. https://reviewboard.mozilla.org/r/59274/#review56364
Attachment #8762799 - Flags: review?(giles) → review+
Attachment #8762800 - Flags: review?(giles) → review+
Comment on attachment 8762800 [details] Bug 1279865 - Don't run TimeMarchesOn if the MediaElement is not played. https://reviewboard.mozilla.org/r/59276/#review56368 It is better to have the first line of the commit message be a short summary of *what* is changed in the patch, with details on separate lines below.
Comment on attachment 8762799 [details] Bug 1279865 - Enable html/semantics/embedded-content/media-elements/interfaces/TextTrack/activeCues.html. Review request updated; see interdiff: https://reviewboard.mozilla.org/r/59274/diff/1-2/
Attachment #8762800 - Attachment description: Bug 1279865 - 1. If mHasUserInteraction MediaElement is false, don't run the TimeMarchesOn because the element is not played. 2. Update the activeCueList only in TimeMarchesOn(). 3. run TimeMarchesOn() at the beginning of play. → Bug 1279865 - Don't run TimeMarchesOn if the MediaElement is not played.
Comment on attachment 8762800 [details] Bug 1279865 - Don't run TimeMarchesOn if the MediaElement is not played. Review request updated; see interdiff: https://reviewboard.mozilla.org/r/59276/diff/1-2/
Comment on attachment 8762799 [details] Bug 1279865 - Enable html/semantics/embedded-content/media-elements/interfaces/TextTrack/activeCues.html. Review request updated; see interdiff: https://reviewboard.mozilla.org/r/59274/diff/2-3/
Comment on attachment 8762800 [details] Bug 1279865 - Don't run TimeMarchesOn if the MediaElement is not played. Review request updated; see interdiff: https://reviewboard.mozilla.org/r/59276/diff/2-3/
https://reviewboard.mozilla.org/r/59276/#review58090 ::: dom/html/TextTrackManager.cpp:542 (Diff revision 3) > if (mMediaElement && > - (!(mMediaElement->GetPlayedOrSeeked()) || mMediaElement->Seeking())) { > + (!(mMediaElement->GetPlayedOrSeeked()) || mMediaElement->Seeking() > + || !mMediaElement->GetHasUserInteraction())) { > return; I miss the autoplay case, so the test_webvtt_empty_displaystate.html failed.
Comment on attachment 8762799 [details] Bug 1279865 - Enable html/semantics/embedded-content/media-elements/interfaces/TextTrack/activeCues.html. Review request updated; see interdiff: https://reviewboard.mozilla.org/r/59274/diff/3-4/
Comment on attachment 8762800 [details] Bug 1279865 - Don't run TimeMarchesOn if the MediaElement is not played. Review request updated; see interdiff: https://reviewboard.mozilla.org/r/59276/diff/3-4/
Comment on attachment 8762800 [details] Bug 1279865 - Don't run TimeMarchesOn if the MediaElement is not played. Review request updated; see interdiff: https://reviewboard.mozilla.org/r/59276/diff/4-5/
Keywords: checkin-needed
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/add8122d11a8 Enable html/semantics/embedded-content/media-elements/interfaces/TextTrack/activeCues.html. r=rillian https://hg.mozilla.org/integration/mozilla-inbound/rev/57cf7cae92f1 Don't run TimeMarchesOn if the MediaElement is not played. r=rillian
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/add8122d11a8 https://hg.mozilla.org/mozilla-central/rev/57cf7cae92f1
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: