1204757 - Remove MediaDataDemuxer::Init WAITING_FOR_DATA failure

Status: RESOLVED FIXED

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

Description

[Jean-Yves Avenard [:jya]]

It is expected that the MediaFormatReader will call MediaDataDemuxer::Init() should the demuxer failed to resolve the init promise.

However, this is only a scenario that works using MediaSource.

Instead, the MediaDataDemuxer should not resolve the promise until it has all the metadata and reject it if having partial metadata is an error (this is the case for all demuxers but the mediasource one)

Comment 1

[Jean-Yves Avenard [:jya]]

Attachment: P1. Update MediaDataDemuxer::Init() behavior.

A MediaDataDemuxer is now not to resolve the init promise until it has all the metadata.
Except MediaSource, all demuxers would be doing blocking read to scan for the metadata, and having partial metadata would be an error.
For MediaSource, we pass the NotifyDataArrived message which will cause the MediaSourceDemuxer to re-attempt to search for the metadata.

When used within MediaSource's TrackBuffersManager, a demuxer will never be created until we have received a complete init segment (this task is performed by the various ContainerParsers)

Comment 2

[Jean-Yves Avenard [:jya]]

Attachment: [MSE] P2. Remove use of MediaDecoder::NotifyWaitingForResourcesStatusChanged().

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8661037 [details] [diff] [review]
P1. Update MediaDataDemuxer::Init() behavior.

Review of attachment 8661037 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/media/MediaDataDemuxer.h
@@ +45,5 @@
>  
>    // Initializes the demuxer. Other methods cannot be called unless
>    // initialization has completed and succeeded.
>    // Typically a demuxer will wait to parse the metadata before resolving the
> +  // promise. The promise must not be resolved should insufficient data be

Perhaps it's better to say the promise won't be resolved until sufficient data is supplied, i.e. your comment here could be read as the promise is *never* resolved if this is called when insufficient data is available.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a3a0f4f5b416
https://hg.mozilla.org/integration/mozilla-inbound/rev/cc917807f8ae

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/a3a0f4f5b416
https://hg.mozilla.org/mozilla-central/rev/cc917807f8ae