1787567 - Looping fails to work for media engine playback

Status: RESOLVED FIXED

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

Comment 1

[Alastor Wu [:alwu]]

Attachment: Bug 1787567 - make looping work for the media engine playback.

This patch fixes several looping issues by doing following changes

1. always store the sample request tokens

When a sample request comes after a stream ends, if we don't store the
token, then we won't be able to send the sample to the media engine
because there is no sample request token. The media engine is still
waiting for the reply for the previous tokens which weren't stored by
us which causing the playback stuck.

2. check sample request token after receiving EOS

If a sample request comes before EOS, then we should still reply that
after receiving EOS. Otherwise, same as (1), the media engine will still
be waiting that token

3. prevent sending end events multiple times
   Sending MEEndOfStream multiple times causes the media engine sending
   MF_MEDIA_ENGINE_EVENT_ENDED multiple times as well, which causes some
   unnessary code execution.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bbbdff6fe2f5
make looping work for the media engine playback. r=jolin

Comment 3

[Atila Butkovits]

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