1047204 - [EME] Implement GMPAudioDecoder interface

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

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

Implement the GMPAudioDecoder interface to allow using GMPs to decode audio.

Comment 1

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

Attachment: Patch v1: Implement GMPAudioDecoder

Basically copy the trail blazed by GMPVideoDecoder...

Comment 2

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 8465955 [details] [diff] [review]
Patch v1: Implement GMPAudioDecoder

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

::: content/media/gmp/GMPAudioDecoderChild.cpp
@@ +100,5 @@
> +  x.mChannelCount = a.mChannelCount();
> +  x.mBitsPerChannel = a.mBitsPerChannel();
> +  x.mSamplesPerSecond = a.mSamplesPerSecond();
> +  x.mExtraData = a.mExtraData().Elements();
> +  x.mExtraDataLen = a.mExtraData().Length();

more descriptive variable names wouldn't be a bad idea.... characters are cheaper than they were 30 years ago

@@ +158,5 @@
> +    mAudioDecoder->DecodingComplete();
> +    mAudioDecoder = nullptr;
> +  }
> +
> +  //mAudioHost.DoneWithAPI();

remove

::: content/media/gmp/GMPService.cpp
@@ +244,5 @@
>  NS_IMETHODIMP
> +GeckoMediaPluginService::GetGMPAudioDecoder(nsTArray<nsCString>* aTags,
> +                                            const nsAString& aOrigin,
> +                                            GMPAudioDecoderProxy** aGMPAD)
> +{

It seems as if these GetGMPFoo's could all be one function.... or at least a template

::: content/media/gmp/PGMPAudioDecoder.ipdl
@@ +21,5 @@
> +  InitDecode(GMPAudioCodecData aCodecSettings);
> +  Decode(GMPAudioEncodedSampleData aInput);
> +  Reset();
> +  Drain();
> +  DecodingComplete(); 

spaces

::: content/media/gmp/mozIGeckoMediaPluginService.idl
@@ +62,5 @@
>    [noscript]
>    GMPVideoEncoderProxy getGMPVideoEncoder(in TagArray tags,
>  		                                      [optional] in AString origin,
>  		                                      out GMPVideoHost outVideoHost);
> +                                          

spaces

Comment 3

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

https://hg.mozilla.org/integration/mozilla-inbound/rev/2bdb11379d0f

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/2bdb11379d0f