1282142 - [EME]: navigator.requestMediaKeySystemAccess should fail for unsupported session types for clear key

Status: RESOLVED FIXED

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

Description

[Mark Watson]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.103 Safari/537.36

Steps to reproduce:

navigator.requestMediaKeySystemAccess( 'org.w3.clearkey', [ { sessionTypes: [ 'persistent-usage-record' ] } ] )


Actual results:

The returned promise resolves with a MediaKeyAccess object from which a MediaKeys can be created and then on createSession( 'persistent-usage-record') an error is thrown.


Expected results:

The requestMediaKeySystemAccess should throw NotSupportedException per the specification https://w3c.github.io/encrypted-media/#get-supported-configuration-and-consent, step 13.3

Comment 1

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

Mass change P2 -> P3

Comment 2

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

Support for the latest navigator.requestMediaKeySystemAccess() behaviour (except prompting for consent) is implemented in bug 1278198, which just landed.

I'll add a test case here to enforce expected results above, since our implementation doesn't support persistent-usage-record MediaKeySessionTypes.

Comment 3

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

Attachment: Bug 1282142 - Add testcase ensuring persistent-usage-record MediaKeySessions are unsupported by ClearKey.

Review commit: https://reviewboard.mozilla.org/r/66324/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/66324/

Comment 4

[Gerald Squelart (he/him) (not at Mozilla since 2022-09-15)]

Comment on attachment 8773593 [details]
Bug 1282142 - Add testcase ensuring persistent-usage-record MediaKeySessions are unsupported by ClearKey.

https://reviewboard.mozilla.org/r/66324/#review63102

Comment 5

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

https://treeherder.mozilla.org/#/jobs?repo=try&revision=880d9c972519

Comment 6

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

https://hg.mozilla.org/integration/mozilla-inbound/rev/48bcc4d343b3a33d08b13300f5a8b0d71e793a32
Bug 1282142 - Add testcase ensuring persistent-usage-record MediaKeySessions are unsupported by ClearKey. r=gerald

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/48bcc4d343b3
Add testcase ensuring persistent-usage-record MediaKeySessions are unsupported by ClearKey. r=gerald

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/48bcc4d343b3