Implement checks for distinctiveIdentifier and persistentState in requestMediaKeySystemAccess().

media/base/key_system_info.h

Index: media/base/key_system_info.h
diff --git a/media/base/key_system_info.h b/media/base/key_system_info.h
index e259c00d855c755b397784e6662e8e88800dc7e6..9dde962293e35d2133871e70fae281d034d1e175 100644
--- a/media/base/key_system_info.h
+++ b/media/base/key_system_info.h
@@ -30,8 +30,8 @@ namespace media {
 // Contains information about an EME key system as well as how to instantiate
 // the corresponding CDM.
 struct MEDIA_EXPORT KeySystemInfo {
+  KeySystemInfo();

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

Why do we have a default constructor and no longer a destructor defined?

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

The constructor was added so that KeySystemInfo structs can go in containers
(hash_maps in particular).

The destructor was removed because it didn't do anything.

   explicit KeySystemInfo(const std::string& key_system);
-  ~KeySystemInfo();
 
   std::string key_system;
 
@@ -41,6 +41,11 @@ struct MEDIA_EXPORT KeySystemInfo {
   // Specifies codecs supported by |key_system|.
   SupportedCodecs supported_codecs;
 
+  // Specifies support for distinctiveIdentifier and persistentState.
+  EmeRequirement persistent_state_requirement;

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

Related to earlier: Is it requirement or support?

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

Acknowledged.

+  EmeRequirement distinctive_identifier_requirement;
+  SupportedSessionTypes supported_session_types;
+
   // A hierarchical parent for |key_system|. This value can be used to check
   // supported types but cannot be used to instantiate a MediaKeys object.
   // Only one parent key system is currently supported per concrete key system.