Implement checks for distinctiveIdentifier and persistentState in requestMediaKeySystemAccess().

media/base/key_system_info.h

 // Copyright 2013 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_KEY_SYSTEM_INFO_H_
 #define MEDIA_BASE_KEY_SYSTEM_INFO_H_
 
 #include <string>
 
 #include "media/base/eme_constants.h"
@@ skipping 12 matching lines @@
 // * Parent key system
 //    A key system string that is one level up from the child key system. It may
 //    be an abstract key system.
 //    As an example, "com.example" is the parent of "com.example.foo".
 
 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;
 
   // Specifies registered initialization data types supported by |key_system|.
   SupportedInitDataTypes supported_init_data_types;
 
   // 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.
   std::string parent_key_system;
 
   // The following indicate how the corresponding CDM should be instantiated.
   bool use_aes_decryptor;
 #if defined(ENABLE_PEPPER_CDMS)
   std::string pepper_type;
 #endif
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_KEY_SYSTEM_INFO_H_