1553678 - [webvtt] Enable wpt 'track-active-cues.html'.

Status: RESOLVED FIXED

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

Description

[Alastor Wu [:alwu]]

The reason of the wpt 'track-active-cues.html' being timout is because we didn't run TimeMarchesOn when we add new cues, so that we couldn't receive cuechange event.

According to the spec [1], the TimeMarchesOn should only be ran when media element's show poster flag is not set. However, in this test, we didn't do anything which would unset the show poster. eg. call play, start autoplay or seek.

[1] https://html.spec.whatwg.org/multipage/media.html#playing-the-media-resource:time-marches-on-2

Comment 1

[Alastor Wu [:alwu]]

Attachment: Bug 1553678 - enable wpt 'track-active-cues.html'

According to the spec [1], the TimeMarchesOn should only be ran when media element's show poster flag is not set.

If we would like to receive cuechange event, we have to reset media element's show poster flag in order to run TimeMarchesOn when adding new cues.

[1] https://html.spec.whatwg.org/multipage/media.html#playing-the-media-resource:time-marches-on-2

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c8f80978ad74
enable wpt 'track-active-cues.html' r=jya

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/17040 for changes under testing/web-platform/tests

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 5

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/c8f80978ad74

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged