1016709 - [EME] Shutdown existing MediaKeys if HTMLMediaElement.mediaKeys is non null when set

Status: RESOLVED WONTFIX

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

Description

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

As per Bug 1016162 comment 2, if we set HTMLMediaElement.mediaKeys and it's already non null, we need to either shutdown the existing instance, or throw an error or something.

Comment 1

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

Spec: https://dvcs.w3.org/hg/html-media/raw-file/default/encrypted-media/encrypted-media.html#dom-setmediakeys

We also need to ensure that we associate a MediaKeys instance with a single HTMLMediaElement.

I don't think shutting down the MediaKeys/CDM is the right thing to do here, we should shut it down once the MediaKeys refcount reaches 0 (which may very well happen when HTMLMediaElement.setMediaKeys(null) is called).

Comment 2

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

Some discussion here:
http://www.w3.org/2014/08/19-html-media-minutes.html#item09
https://www.w3.org/Bugs/Public/show_bug.cgi?id=24771

Comment 3

[Chris Peterson [:cpeterson]]

These bugs are necessary for vouching and sandboxing a third-party CDM.

Comment 4

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

Chrome doesn't shutdown the any MediaKeys already set on an HTMLMediaElement when HTMLMediaElement.setMediaKeys() is called again. I filed bug 1233274 to match our behaviour.