1301294 - Remove return values from MediaDataDecoder

Status: RESOLVED FIXED

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

Description

[Matt Woodrow (:mattwoodrow)]

Attachment: remove-media-data-decoder-retvals

These are basically entirely unused, and the few cases where they are can be replaced with a call to Error() on the callback.

Comment 1

[Jean-Yves Avenard [:jya]]

Comment on attachment 8789219 [details] [diff] [review]
remove-media-data-decoder-retvals

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

r=me with two changes made

::: dom/media/platforms/wrappers/H264Converter.cpp
@@ +54,5 @@
>  {
>    if (!mp4_demuxer::AnnexB::ConvertSampleToAVCC(aSample)) {
>      // We need AVCC content to be able to later parse the SPS.
>      // This is a no-op if the data is already AVCC.
> +    mCallback->Error(MediaDataDecoderError::FATAL_ERROR);

i think this should be a decode error too

@@ +78,5 @@
>    } else {
>      rv = CheckForSPSChange(aSample);
>    }
> +  if (NS_FAILED(rv)) {
> +    mCallback->Error(MediaDataDecoderError::FATAL_ERROR);

this one shouldn't be a fatal error, but decode error. it could be that the sps was invalid

Comment 2

[Jean-Yves Avenard [:jya]]

Bonus points if documentation of MediaDataDecoder is amended to point out that CallBack::Error must be called whenever an error occurs.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/daea33341cc4
Remove unnecessary nsresult return value from MediaDataDecoder interface. r=jya

Comment 4

[Carsten Book [:Tomcat]]

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