1186375 - Add GMP EME flags for rendering capability and APIs to get this capability.

Status: RESOLVED FIXED

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

Description

[Kilik Kuo [:kikuo] (inactive)]

Attachment: Add GMP EME render flag _ 253975

According to Bug 1146795,
Need to provide a way to retrieve CDM rendering capability for MDSM or MediaDecoderReader.
Add these definition and provide APIs.

One minor add,
Do we need to replace 

#define GMP_EME_CAP_DECRYPT_AND_DECODE_AUDIO

to 

#define GMP_EME_CAP_DECODE_AUDIO ?

Because I assume that GME_EME_CAP_RENDER_AUDIO stands for capabilities of decrypt/decode/render. (To avoid lengthy naming)
But I'm not sure if others have the same thought.

Comment 1

[Kilik Kuo [:kikuo] (inactive)]

Comment on attachment 8637160 [details] [diff] [review]
Add GMP EME render flag _ 253975

Hi Chris, 

Do you have time to review smalle patch ?
I got a f+ from JW in Bug 1146795 Attachment 8631654 [details] [diff], I think it should be ok to be reviewed.

Comment 2

[Kilik Kuo [:kikuo] (inactive)]

Thanks Chris ! 


try result looks okay
https://treeherder.mozilla.org/#/jobs?repo=try&revision=faf5fd880bab

Comment 3

[Kilik Kuo [:kikuo] (inactive)]

Thanks Chris ! 


try result looks okay
https://treeherder.mozilla.org/#/jobs?repo=try&revision=faf5fd880bab

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a5b0d78839c0

Comment 5

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

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