Implement checks for distinctiveIdentifier and persistentState in requestMediaKeySystemAccess().

media/base/eme_constants.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_BASE_EME_CONSTANTS_H_
 #define MEDIA_BASE_EME_CONSTANTS_H_
 
 #include <stdint.h>
 
 namespace media {
@@ skipping 28 matching lines @@
   EME_CODEC_MP4_AUDIO_ALL = EME_CODEC_MP4_AAC,
   EME_CODEC_MP4_AVC1 = 1 << 5,
   EME_CODEC_MP4_VIDEO_ALL = EME_CODEC_MP4_AVC1,
   EME_CODEC_MP4_ALL = (EME_CODEC_MP4_AUDIO_ALL | EME_CODEC_MP4_VIDEO_ALL),
   EME_CODEC_ALL = (EME_CODEC_WEBM_ALL | EME_CODEC_MP4_ALL),
 #else
   EME_CODEC_ALL = EME_CODEC_WEBM_ALL,
 #endif  // defined(USE_PROPRIETARY_CODECS)
 };
 
+enum EmeSessionType {
+  EME_SESSION_TYPE_NONE = 0,
+  EME_SESSION_TYPE_TEMPORARY = 1 << 0,
+  EME_SESSION_TYPE_PERSISTENT_LICENSE = 1 << 1,
+  EME_SESSION_TYPE_PERSISTENT_RELEASE_MESSAGE = 1 << 2,
+};
+
 typedef uint32_t SupportedInitDataTypes;
 typedef uint32_t SupportedCodecs;
+typedef uint32_t SupportedSessionTypes;
+
+enum EmeRequirement {
+  EME_REQUIREMENT_NOT_ALLOWED = 1 << 0,

[ddorwin, 2015/02/17 20:56:47]

Would it make sense (and help with the code) to have a different set of values
when specifying key system capabilities? For example, SUPPORTS_NOT_ALLOWED,
REQUIRES_ALWAYS, WILL_USE_IF_ALLOWED.

[sandersd (OOO until July 31), 2015/02/18 23:41:17]

Done.

+  EME_REQUIREMENT_REQUIRED = 1 << 1,
+  EME_REQUIREMENT_OPTIONAL =

[ddorwin, 2015/02/17 20:56:47]

Embedders should never use this, right? Maybe it shouldn't be in this file. I'm
not sure "OPTIONAL" here has the same meaning as the spec - it's really just
"supports both", which we probably don't need. Also, see above.

[sandersd (OOO until July 31), 2015/02/18 23:41:17]

Acknowledged.

+      EME_REQUIREMENT_NOT_ALLOWED | EME_REQUIREMENT_REQUIRED,
+};
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_EME_CONSTANTS_H_