Extract requestMediaKeySystemAccess() algorithm.

media/blink/key_system_config_selector.cc

Index: media/blink/key_system_config_selector.cc
diff --git a/media/blink/webencryptedmediaclient_impl.cc b/media/blink/key_system_config_selector.cc
similarity index 72%
copy from media/blink/webencryptedmediaclient_impl.cc
copy to media/blink/key_system_config_selector.cc
index faa7657f4cd81cf393940635b707e6c44bb659af..c89c15240c34271933f15eb72546bd553fd34e30 100644
--- a/media/blink/webencryptedmediaclient_impl.cc
+++ b/media/blink/key_system_config_selector.cc
@@ -2,43 +2,62 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "webencryptedmediaclient_impl.h"
+#include "key_system_config_selector.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
-#include "base/metrics/histogram.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "media/base/key_systems.h"
 #include "media/base/media_permission.h"
-#include "media/blink/webcontentdecryptionmodule_impl.h"
-#include "media/blink/webcontentdecryptionmoduleaccess_impl.h"
 #include "media/blink/webmediaplayer_util.h"
 #include "net/base/mime_util.h"
-#include "third_party/WebKit/public/platform/WebEncryptedMediaRequest.h"
 #include "third_party/WebKit/public/platform/WebMediaKeySystemConfiguration.h"
+#include "third_party/WebKit/public/platform/WebSecurityOrigin.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 #include "third_party/WebKit/public/platform/WebVector.h"
+#include "url/gurl.h"
 
 namespace media {
 
-// These names are used by UMA.
-const char kKeySystemSupportUMAPrefix[] =
-    "Media.EME.RequestMediaKeySystemAccess.";
+namespace {
 
-enum ConfigurationSupport {
-  CONFIGURATION_NOT_SUPPORTED,
-  CONFIGURATION_REQUIRES_PERMISSION,
-  CONFIGURATION_SUPPORTED,
+static EmeFeatureRequirement ConvertRequirement(
+    blink::WebMediaKeySystemConfiguration::Requirement requirement) {
+  switch (requirement) {
+    case blink::WebMediaKeySystemConfiguration::Requirement::Required:
+      return EME_FEATURE_REQUIRED;
+    case blink::WebMediaKeySystemConfiguration::Requirement::Optional:
+      return EME_FEATURE_OPTIONAL;
+    case blink::WebMediaKeySystemConfiguration::Requirement::NotAllowed:
+      return EME_FEATURE_NOT_ALLOWED;
+  }
+
+  NOTREACHED();
+  return EME_FEATURE_NOT_ALLOWED;
+}
+
+}  // namespace
+
+struct KeySystemConfigSelector::SelectionRequest {
+  std::string key_system;
+  blink::WebVector<blink::WebMediaKeySystemConfiguration>
+      candidate_configurations;
+  blink::WebSecurityOrigin security_origin;
+  base::Callback<void(const blink::WebMediaKeySystemConfiguration&)>
+      succeeded_cb;
+  base::Callback<void(const blink::WebString&)> not_supported_cb;
+  bool was_permission_requested = false;
+  bool is_permission_granted = false;
 };
 
 // Accumulates configuration rules to determine if a feature (additional
 // configuration rule) can be added to an accumulated configuration.
-class ConfigState {
+class KeySystemConfigSelector::ConfigState {
  public:
-  ConfigState(bool was_permission_requested, bool is_permission_granted)
-      : was_permission_requested_(was_permission_requested),
-        is_permission_granted_(is_permission_granted) {
+  ConfigState(const SelectionRequest* request)

[ddorwin, 2015/04/03 22:09:17]

It seems wrong to pass the entire request object here, especially since (I
think) ConfigState is used for a single config and the request contains
multiple. Passing individual bools was better.

If left like this, it would need explicit.

[sandersd (OOO until July 31), 2015/04/03 22:44:57]

Done.

+      : was_permission_requested_(request->was_permission_requested),
+        is_permission_granted_(request->is_permission_granted) {
   }
 
   bool IsPermissionGranted() const {
@@ -135,143 +154,119 @@ class ConfigState {
   // Whether a rule has been added that requires or blocks persistent state.
   bool is_persistence_required_ = false;
   bool is_persistence_not_allowed_ = false;
+
+  DISALLOW_COPY_AND_ASSIGN(ConfigState);
 };
 
-static bool IsSupportedContentType(
-    const KeySystems& key_systems,
-    const std::string& key_system,
-    EmeMediaType media_type,
-    const std::string& container_mime_type,
-    const std::string& codecs) {
-  // TODO(sandersd): Move contentType parsing from Blink to here so that invalid
-  // parameters can be rejected. http://crbug.com/417561
-  std::string container_lower = base::StringToLowerASCII(container_mime_type);
+KeySystemConfigSelector::KeySystemConfigSelector(
+    const KeySystems* key_systems,
+    MediaPermission* media_permission)
+    : key_systems_(key_systems),
+      media_permission_(media_permission),
+      weak_factory_(this) {
+  DCHECK(key_systems_);
+  DCHECK(media_permission_);
+}
 
-  // Check that |container_mime_type| and |codecs| are supported by the CDM.
-  // This check does not handle extended codecs, so extended codec information
-  // is stripped.
-  std::vector<std::string> codec_vector;
-  net::ParseCodecString(codecs, &codec_vector, true);
-  if (!key_systems.IsSupportedCodecCombination(
-           key_system, media_type, container_lower, codec_vector)) {
-    return false;
+KeySystemConfigSelector::~KeySystemConfigSelector() {
+}
+
+void KeySystemConfigSelector::SelectConfig(
+    const blink::WebString& key_system,
+    const blink::WebVector<blink::WebMediaKeySystemConfiguration>&
+        candidate_configurations,
+    const blink::WebSecurityOrigin& security_origin,
+    base::Callback<void(const blink::WebMediaKeySystemConfiguration&)>
+        succeeded_cb,
+    base::Callback<void(const blink::WebString&)> not_supported_cb) {
+  // Continued from requestMediaKeySystemAccess(), step 7, from
+  // https://w3c.github.io/encrypted-media/#requestmediakeysystemaccess
+  //
+  // 7.1. If keySystem is not one of the Key Systems supported by the user
+  //      agent, reject promise with with a new DOMException whose name is
+  //      NotSupportedError. String comparison is case-sensitive.
+  if (!base::IsStringASCII(key_system)) {
+    not_supported_cb.Run("Only ASCII keySystems are supported");
+    return;
   }
 
-  // Check that |container_mime_type| is supported by Chrome. This can only
-  // happen if the CDM declares support for a container that Chrome does not.
-  if (!net::IsSupportedMediaMimeType(container_lower)) {
-    NOTREACHED();
-    return false;
+  scoped_ptr<SelectionRequest> request(new SelectionRequest());

[ddorwin, 2015/04/03 22:09:17]

nit: It seems wrong to create an object to check the string. Normally we would
create when we no we'll succeed. The compiler can probably avoid the copy.

[sandersd (OOO until July 31), 2015/04/03 22:44:57]

Done.

+  request->key_system = base::UTF16ToASCII(key_system);
+  if (!key_systems_->IsSupportedKeySystem(request->key_system)) {
+    not_supported_cb.Run("Unsupported keySystem");
+    return;
   }
 
-  // Check that |codecs| are supported by Chrome. This is done primarily to
-  // validate extended codecs, but it also ensures that the CDM cannot support
-  // codecs that Chrome does not (which could complicate the robustness
-  // algorithm).
-  if (codec_vector.empty())
-    return true;
-  codec_vector.clear();
-  net::ParseCodecString(codecs, &codec_vector, false);
-  return (net::IsSupportedStrictMediaMimeType(container_lower, codec_vector) ==
-          net::IsSupported);
+  // 7.2-7.4. Implemented by OnSelectConfig().
+  // TODO(sandersd): This should be async, ideally not on the main thread.
+  request->candidate_configurations = candidate_configurations;
+  request->security_origin = security_origin;
+  request->succeeded_cb = succeeded_cb;
+  request->not_supported_cb = not_supported_cb;
+  OnSelectConfig(request.Pass());

[ddorwin, 2015/04/03 22:09:17]

nit: "On" doesn't seem appropriate, even if this was posted.

The same old naming debate: Do..., ...Internal, etc.

[sandersd (OOO until July 31), 2015/04/03 22:44:57]

Done.

 }
 
-static bool GetSupportedCapabilities(
-    const KeySystems& key_systems,
-    const std::string& key_system,
-    EmeMediaType media_type,
-    const blink::WebVector<blink::WebMediaKeySystemMediaCapability>&
-        requested_media_capabilities,
-    ConfigState* config_state,
-    std::vector<blink::WebMediaKeySystemMediaCapability>*
-        supported_media_capabilities) {
-  // From
-  // https://w3c.github.io/encrypted-media/#get-supported-capabilities-for-media-type
-  // 1. Let local accumulated capabilities be a local copy of partial
-  //    configuration.
-  //    (Skipped as we directly update |config_state|. This is safe because we
-  //    only do so when at least one requested media capability is supported.)
-  // 2. Let supported media capabilities be empty.
-  DCHECK_EQ(supported_media_capabilities->size(), 0ul);
-  // 3. For each value in requested media capabilities:
-  for (size_t i = 0; i < requested_media_capabilities.size(); i++) {
-    // 3.1. Let contentType be the value's contentType member.
-    // 3.2. Let robustness be the value's robustness member.
-    const blink::WebMediaKeySystemMediaCapability& capability =
-        requested_media_capabilities[i];
-    // 3.3. If contentType is the empty string, return null.
-    if (capability.mimeType.isEmpty()) {
-      DVLOG(2) << "Rejecting requested configuration because "
-               << "a capability contentType was empty.";
-      return false;
-    }
-    // 3.4-3.11. (Implemented by IsSupportedContentType().)
-    if (!base::IsStringASCII(capability.mimeType) ||
-        !base::IsStringASCII(capability.codecs) ||
-        !IsSupportedContentType(key_systems, key_system, media_type,
-                                base::UTF16ToASCII(capability.mimeType),
-                                base::UTF16ToASCII(capability.codecs))) {
-      continue;
-    }
-    // 3.12. If robustness is not the empty string, run the following steps:
-    if (!capability.robustness.isEmpty()) {
-      // 3.12.1. If robustness is an unrecognized value or not supported by
-      //         implementation, continue to the next iteration. String
-      //         comparison is case-sensitive.
-      if (!base::IsStringASCII(capability.robustness))
-        continue;
-      EmeConfigRule robustness_rule = key_systems.GetRobustnessConfigRule(
-          key_system, media_type, base::UTF16ToASCII(capability.robustness));
-      if (!config_state->IsRuleSupported(robustness_rule))
-        continue;
-      config_state->AddRule(robustness_rule);
-      // 3.12.2. Add robustness to configuration.
-      //         (It's already added, we use capability as configuration.)
-    }
-    // 3.13. If the user agent and implementation do not support playback of
-    //       encrypted media data as specified by configuration, including all
-    //       media types, in combination with local accumulated capabilities,
-    //       continue to the next iteration.
-    //       (This is handled when adding rules to |config_state|.)
-    // 3.14. Add configuration to supported media capabilities.
-    supported_media_capabilities->push_back(capability);
-    // 3.15. Add configuration to local accumulated capabilities.
-    //       (Skipped as we directly update |config_state|.)
-  }
-  // 4. If supported media capabilities is empty, return null.
-  if (supported_media_capabilities->empty()) {
-    DVLOG(2) << "Rejecting requested configuration because "
-             << "no capabilities were supported.";
-    return false;
-  }
-  // 5. Return media type capabilities.
-  return true;
+void KeySystemConfigSelector::OnPermissionResult(
+    scoped_ptr<SelectionRequest> request,
+    bool is_permission_granted) {
+  request->was_permission_requested = true;
+  request->is_permission_granted = is_permission_granted;
+  OnSelectConfig(request.Pass());
 }
 
-static EmeFeatureRequirement ConvertRequirement(
-    blink::WebMediaKeySystemConfiguration::Requirement requirement) {
-  switch (requirement) {
-    case blink::WebMediaKeySystemConfiguration::Requirement::Required:
-      return EME_FEATURE_REQUIRED;
-    case blink::WebMediaKeySystemConfiguration::Requirement::Optional:
-      return EME_FEATURE_OPTIONAL;
-    case blink::WebMediaKeySystemConfiguration::Requirement::NotAllowed:
-      return EME_FEATURE_NOT_ALLOWED;
+void KeySystemConfigSelector::OnSelectConfig(
+    scoped_ptr<SelectionRequest> request) {
+  // Continued from requestMediaKeySystemAccess(), step 7.1, from
+  // https://w3c.github.io/encrypted-media/#requestmediakeysystemaccess
+  //
+  // 7.2. Let implementation be the implementation of keySystem.
+  //      (|key_systems_| fills this role.)
+  // 7.3. For each value in supportedConfigurations:
+  for (size_t i = 0; i < request->candidate_configurations.size(); i++) {
+    // 7.3.1. Let candidate configuration be the value.
+    // 7.3.2. Let supported configuration be the result of executing the Get
+    //        Supported Configuration algorithm on implementation, candidate
+    //        configuration, and origin.
+    // 7.3.3. If supported configuration is not null, [initialize and return a
+    //        new MediaKeySystemAccess object.]
+    ConfigState config_state(request.get());

[ddorwin, 2015/04/03 22:09:17]

Why is ConfigState passed in? It's not used outside GetSupportedConfiguration().

For testing?

[sandersd (OOO until July 31), 2015/04/03 22:44:57]

Two reasons:
  1) It's clearer than passing two bools.
  2) In the future we will want to pull some bits out of the config state (eg.
for Android SetSecurityLevel()).

+    blink::WebMediaKeySystemConfiguration accumulated_configuration;
+    ConfigurationSupport support = GetSupportedConfiguration(
+        request->key_system, request->candidate_configurations[i],
+        &config_state, &accumulated_configuration);
+    switch (support) {
+      case CONFIGURATION_NOT_SUPPORTED:
+        continue;
+      case CONFIGURATION_REQUIRES_PERMISSION:
+        if (request->was_permission_requested) {
+          DVLOG(2) << "Rejecting requested configuration because "
+                   << "permission was denied.";
+          continue;
+        }
+        media_permission_->RequestPermission(
+            MediaPermission::PROTECTED_MEDIA_IDENTIFIER,
+            GURL(request->security_origin.toString()),
+            base::Bind(&KeySystemConfigSelector::OnPermissionResult,
+                       weak_factory_.GetWeakPtr(), base::Passed(&request)));
+        return;
+      case CONFIGURATION_SUPPORTED:
+        request->succeeded_cb.Run(accumulated_configuration);
+        return;
+    }
   }
 
-  NOTREACHED();
-  return EME_FEATURE_NOT_ALLOWED;
+  // 7.4. Reject promise with a new DOMException whose name is
+  //      NotSupportedError.
+  request->not_supported_cb.Run(
+      "None of the requested configurations were supported.");
 }
 
-static ConfigurationSupport GetSupportedConfiguration(
-    const KeySystems& key_systems,
+KeySystemConfigSelector::ConfigurationSupport
+KeySystemConfigSelector::GetSupportedConfiguration(
     const std::string& key_system,
     const blink::WebMediaKeySystemConfiguration& candidate,
-    bool was_permission_requested,
-    bool is_permission_granted,
+    ConfigState* config_state,
     blink::WebMediaKeySystemConfiguration* accumulated_configuration) {
-  ConfigState config_state(was_permission_requested, is_permission_granted);
-
   // From https://w3c.github.io/encrypted-media/#get-supported-configuration
   // 1. Let accumulated configuration be empty. (Done by caller.)
   // 2. If the initDataTypes member is present in candidate configuration, run
@@ -289,9 +284,7 @@ static ConfigurationSupport GetSupportedConfiguration(
       //        initDataType, add initDataType to supported types. String
       //        comparison is case-sensitive. The empty string is never
       //        supported.
-      if (init_data_type == blink::WebEncryptedMediaInitDataType::Unknown)
-        continue;
-      if (key_systems.IsSupportedInitDataType(
+      if (key_systems_->IsSupportedInitDataType(
               key_system, ConvertToEmeInitDataType(init_data_type))) {
         supported_types.push_back(init_data_type);
       }
@@ -309,7 +302,7 @@ static ConfigurationSupport GetSupportedConfiguration(
   }
 
   // 3. Follow the steps for the value of candidate configuration's
-  //    distinctiveIdentifier member from the following list:
+  //    distinctiveIdentifier attribute from the following list:

[ddorwin, 2015/04/03 22:09:18]

regression in use of "member"? Below as well.

[sandersd (OOO until July 31), 2015/04/03 22:44:57]

Done.

   //      - "required": If the implementation does not support a persistent
   //        Distinctive Identifier in combination with accumulated
   //        configuration, return null.
@@ -319,38 +312,38 @@ static ConfigurationSupport GetSupportedConfiguration(
   //        null.
   // We also reject OPTIONAL when distinctive identifiers are ALWAYS_ENABLED and
   // permission has already been denied. This would happen anyway at step 11.
-  EmeConfigRule di_rule = key_systems.GetDistinctiveIdentifierConfigRule(
+  EmeConfigRule di_rule = key_systems_->GetDistinctiveIdentifierConfigRule(
       key_system, ConvertRequirement(candidate.distinctiveIdentifier));
-  if (!config_state.IsRuleSupported(di_rule)) {
+  if (!config_state->IsRuleSupported(di_rule)) {
     DVLOG(2) << "Rejecting requested configuration because "
              << "the distinctiveIdentifier requirement was not supported.";
     return CONFIGURATION_NOT_SUPPORTED;
   }
-  config_state.AddRule(di_rule);
+  config_state->AddRule(di_rule);
 
   // 4. Add the value of the candidate configuration's distinctiveIdentifier
-  //    member to accumulated configuration.
+  //    attribute to accumulated configuration.
   accumulated_configuration->distinctiveIdentifier =
       candidate.distinctiveIdentifier;
 
   // 5. Follow the steps for the value of candidate configuration's
-  //    persistentState member from the following list:
+  //    persistentState attribute from the following list:
   //      - "required": If the implementation does not support persisting state
   //        in combination with accumulated configuration, return null.
   //      - "optional": Continue.
   //      - "not-allowed": If the implementation requires persisting state in
   //        combination with accumulated configuration, return null.
-  EmeConfigRule ps_rule = key_systems.GetPersistentStateConfigRule(
+  EmeConfigRule ps_rule = key_systems_->GetPersistentStateConfigRule(
       key_system, ConvertRequirement(candidate.persistentState));
-  if (!config_state.IsRuleSupported(ps_rule)) {
+  if (!config_state->IsRuleSupported(ps_rule)) {
     DVLOG(2) << "Rejecting requested configuration because "
              << "the persistentState requirement was not supported.";
     return CONFIGURATION_NOT_SUPPORTED;
   }
-  config_state.AddRule(ps_rule);
+  config_state->AddRule(ps_rule);
 
   // 6. Add the value of the candidate configuration's persistentState
-  //    member to accumulated configuration.
+  //    attribute to accumulated configuration.
   accumulated_configuration->persistentState = candidate.persistentState;
 
   // 7. Follow the steps for the first matching condition from the following
@@ -391,20 +384,20 @@ static ConfigurationSupport GetSupportedConfiguration(
         break;
       case blink::WebEncryptedMediaSessionType::PersistentLicense:
         session_type_rule =
-            key_systems.GetPersistentLicenseSessionConfigRule(key_system);
+            key_systems_->GetPersistentLicenseSessionConfigRule(key_system);
         break;
       case blink::WebEncryptedMediaSessionType::PersistentReleaseMessage:
         session_type_rule =
-            key_systems.GetPersistentReleaseMessageSessionConfigRule(
+            key_systems_->GetPersistentReleaseMessageSessionConfigRule(
                 key_system);
         break;
     }
-    if (!config_state.IsRuleSupported(session_type_rule)) {
+    if (!config_state->IsRuleSupported(session_type_rule)) {
       DVLOG(2) << "Rejecting requested configuration because "
                << "a required session type was not supported.";
       return CONFIGURATION_NOT_SUPPORTED;
     }
-    config_state.AddRule(session_type_rule);
+    config_state->AddRule(session_type_rule);
   }
 
   // 9. Add session types to accumulated configuration.
@@ -418,9 +411,9 @@ static ConfigurationSupport GetSupportedConfiguration(
     //       configuration.
     // 10.2. If video capabilities is null, return null.
     std::vector<blink::WebMediaKeySystemMediaCapability> video_capabilities;
-    if (!GetSupportedCapabilities(key_systems, key_system, EmeMediaType::VIDEO,
+    if (!GetSupportedCapabilities(key_system, EmeMediaType::VIDEO,
                                   candidate.videoCapabilities,
-                                  &config_state, &video_capabilities)) {
+                                  config_state, &video_capabilities)) {
       return CONFIGURATION_NOT_SUPPORTED;
     }
 
@@ -436,9 +429,9 @@ static ConfigurationSupport GetSupportedConfiguration(
     //       configuration.
     // 11.2. If audio capabilities is null, return null.
     std::vector<blink::WebMediaKeySystemMediaCapability> audio_capabilities;
-    if (!GetSupportedCapabilities(key_systems, key_system, EmeMediaType::AUDIO,
+    if (!GetSupportedCapabilities(key_system, EmeMediaType::AUDIO,
                                   candidate.audioCapabilities,
-                                  &config_state, &audio_capabilities)) {
+                                  config_state, &audio_capabilities)) {
       return CONFIGURATION_NOT_SUPPORTED;
     }
 
@@ -457,28 +450,29 @@ static ConfigurationSupport GetSupportedConfiguration(
   if (accumulated_configuration->distinctiveIdentifier ==
       blink::WebMediaKeySystemConfiguration::Requirement::Optional) {
     EmeConfigRule not_allowed_rule =
-        key_systems.GetDistinctiveIdentifierConfigRule(
+        key_systems_->GetDistinctiveIdentifierConfigRule(
             key_system, EME_FEATURE_NOT_ALLOWED);
     EmeConfigRule required_rule =
-        key_systems.GetDistinctiveIdentifierConfigRule(
+        key_systems_->GetDistinctiveIdentifierConfigRule(
             key_system, EME_FEATURE_REQUIRED);
-    bool not_allowed_supported = config_state.IsRuleSupported(not_allowed_rule);
-    bool required_supported = config_state.IsRuleSupported(required_rule);
+    bool not_allowed_supported =
+        config_state->IsRuleSupported(not_allowed_rule);
+    bool required_supported = config_state->IsRuleSupported(required_rule);
     // If a distinctive identifier is recommend and that is a possible outcome,
     // prefer that.
     if (required_supported &&
-        config_state.IsIdentifierRecommended() &&
-        config_state.IsPermissionPossible()) {
+        config_state->IsIdentifierRecommended() &&
+        config_state->IsPermissionPossible()) {
       not_allowed_supported = false;
     }
     if (not_allowed_supported) {
       accumulated_configuration->distinctiveIdentifier =
           blink::WebMediaKeySystemConfiguration::Requirement::NotAllowed;
-      config_state.AddRule(not_allowed_rule);
+      config_state->AddRule(not_allowed_rule);
     } else if (required_supported) {
       accumulated_configuration->distinctiveIdentifier =
           blink::WebMediaKeySystemConfiguration::Requirement::Required;
-      config_state.AddRule(required_rule);
+      config_state->AddRule(required_rule);
     } else {
       // We should not have passed step 3.
       NOTREACHED();
@@ -497,10 +491,10 @@ static ConfigurationSupport GetSupportedConfiguration(
   if (accumulated_configuration->persistentState ==
       blink::WebMediaKeySystemConfiguration::Requirement::Optional) {
     EmeConfigRule not_allowed_rule =
-        key_systems.GetPersistentStateConfigRule(
+        key_systems_->GetPersistentStateConfigRule(
             key_system, EME_FEATURE_NOT_ALLOWED);
     EmeConfigRule required_rule =
-        key_systems.GetPersistentStateConfigRule(
+        key_systems_->GetPersistentStateConfigRule(
             key_system, EME_FEATURE_REQUIRED);
     // |distinctiveIdentifier| should not be affected after it is decided.
     DCHECK(not_allowed_rule == EmeConfigRule::NOT_SUPPORTED ||
@@ -508,17 +502,17 @@ static ConfigurationSupport GetSupportedConfiguration(
     DCHECK(required_rule == EmeConfigRule::NOT_SUPPORTED ||
            required_rule == EmeConfigRule::PERSISTENCE_REQUIRED);
     bool not_allowed_supported =
-        config_state.IsRuleSupported(not_allowed_rule);
+        config_state->IsRuleSupported(not_allowed_rule);
     bool required_supported =
-        config_state.IsRuleSupported(required_rule);
+        config_state->IsRuleSupported(required_rule);
     if (not_allowed_supported) {
       accumulated_configuration->persistentState =
           blink::WebMediaKeySystemConfiguration::Requirement::NotAllowed;
-      config_state.AddRule(not_allowed_rule);
+      config_state->AddRule(not_allowed_rule);
     } else if (required_supported) {
       accumulated_configuration->persistentState =
           blink::WebMediaKeySystemConfiguration::Requirement::Required;
-      config_state.AddRule(required_rule);
+      config_state->AddRule(required_rule);
     } else {
       // We should not have passed step 5.
       NOTREACHED();
@@ -535,196 +529,121 @@ static ConfigurationSupport GetSupportedConfiguration(
       blink::WebMediaKeySystemConfiguration::Requirement::Required) {
     // The caller is responsible for resolving what to do if permission is
     // required but has been denied (it should treat it as NOT_SUPPORTED).
-    if (!config_state.IsPermissionGranted())
+    if (!config_state->IsPermissionGranted())
       return CONFIGURATION_REQUIRES_PERMISSION;
   }
 
-  // 16. If the label member is present in candidate configuration, add the
-  //     value of the candidate configuration's label member to accumulated
-  //     configuration.
-  accumulated_configuration->label = candidate.label;
-
-  // 17. Return accumulated configuration.
+  // 16. Return accumulated configuration.
   return CONFIGURATION_SUPPORTED;
 }
 
-// Report usage of key system to UMA. There are 2 different counts logged:
-// 1. The key system is requested.
-// 2. The requested key system and options are supported.
-// Each stat is only reported once per renderer frame per key system.
-// Note that WebEncryptedMediaClientImpl is only created once by each
-// renderer frame.
-class WebEncryptedMediaClientImpl::Reporter {
- public:
-  enum KeySystemSupportStatus {
-    KEY_SYSTEM_REQUESTED = 0,
-    KEY_SYSTEM_SUPPORTED = 1,
-    KEY_SYSTEM_SUPPORT_STATUS_COUNT
-  };
-
-  explicit Reporter(const std::string& key_system_for_uma)
-      : uma_name_(kKeySystemSupportUMAPrefix + key_system_for_uma),
-        is_request_reported_(false),
-        is_support_reported_(false) {}
-  ~Reporter() {}
-
-  void ReportRequested() {
-    if (is_request_reported_)
-      return;
-    Report(KEY_SYSTEM_REQUESTED);
-    is_request_reported_ = true;
-  }
-
-  void ReportSupported() {
-    DCHECK(is_request_reported_);
-    if (is_support_reported_)
-      return;
-    Report(KEY_SYSTEM_SUPPORTED);
-    is_support_reported_ = true;
+bool KeySystemConfigSelector::GetSupportedCapabilities(

[ddorwin, 2015/04/03 22:09:17]

It might have been better to leave the order to simplify reviewing. Then move it
separately if you want to.

[sandersd (OOO until July 31), 2015/04/03 22:44:57]

Done.

+    const std::string& key_system,
+    EmeMediaType media_type,
+    const blink::WebVector<blink::WebMediaKeySystemMediaCapability>&
+        requested_media_capabilities,
+    KeySystemConfigSelector::ConfigState* config_state,
+    std::vector<blink::WebMediaKeySystemMediaCapability>*
+        supported_media_capabilities) {
+  // From
+  // https://w3c.github.io/encrypted-media/#get-supported-capabilities-for-media-type
+  // 1. Let local accumulated capabilities be a local copy of partial
+  //    configuration.
+  //    (Skipped as we directly update |config_state|. This is safe because we
+  //    only do so when at least one requested media capability is supported.)
+  // 2. Let supported media capabilities be empty.
+  DCHECK_EQ(supported_media_capabilities->size(), 0ul);
+  // 3. For each value in requested media capabilities:
+  for (size_t i = 0; i < requested_media_capabilities.size(); i++) {
+    // 3.1. Let contentType be the value's contentType member.
+    // 3.2. Let robustness be the value's robustness member.
+    const blink::WebMediaKeySystemMediaCapability& capability =
+        requested_media_capabilities[i];
+    // 3.3. If contentType is the empty string, return null.
+    if (capability.mimeType.isEmpty()) {
+      DVLOG(2) << "Rejecting requested configuration because "
+               << "a capability contentType was empty.";
+      return false;
+    }
+    // 3.4-3.11. (Implemented by IsSupportedContentType().)
+    if (!base::IsStringASCII(capability.mimeType) ||
+        !base::IsStringASCII(capability.codecs) ||
+        !IsSupportedContentType(key_system, media_type,
+                                base::UTF16ToASCII(capability.mimeType),
+                                base::UTF16ToASCII(capability.codecs))) {
+      continue;
+    }
+    // 3.12. If robustness is not the empty string, run the following steps:
+    if (!capability.robustness.isEmpty()) {
+      // 3.12.1. If robustness is an unrecognized value or not supported by
+      //         implementation, continue to the next iteration. String
+      //         comparison is case-sensitive.
+      if (!base::IsStringASCII(capability.robustness))
+        continue;
+      EmeConfigRule robustness_rule = key_systems_->GetRobustnessConfigRule(
+          key_system, media_type, base::UTF16ToASCII(capability.robustness));
+      if (!config_state->IsRuleSupported(robustness_rule))
+        continue;
+      config_state->AddRule(robustness_rule);
+      // 3.12.2. Add robustness to configuration.
+      //         (It's already added, we use capability as configuration.)
+    }
+    // 3.13. If the user agent and implementation do not support playback of
+    //       encrypted media data as specified by configuration, including all
+    //       media types, in combination with local accumulated capabilities,
+    //       continue to the next iteration.
+    //       (This is handled when adding rules to |config_state|.)
+    // 3.14. Add configuration to supported media capabilities.
+    supported_media_capabilities->push_back(capability);
+    // 3.15. Add configuration to local accumulated capabilities.
+    //       (Skipped as we directly update |config_state|.)
   }
-
- private:
-  void Report(KeySystemSupportStatus status) {
-    // Not using UMA_HISTOGRAM_ENUMERATION directly because UMA_* macros
-    // require the names to be constant throughout the process' lifetime.
-    base::LinearHistogram::FactoryGet(
-        uma_name_, 1, KEY_SYSTEM_SUPPORT_STATUS_COUNT,
-        KEY_SYSTEM_SUPPORT_STATUS_COUNT + 1,
-        base::Histogram::kUmaTargetedHistogramFlag)->Add(status);
+  // 4. If supported media capabilities is empty, return null.
+  if (supported_media_capabilities->empty()) {
+    DVLOG(2) << "Rejecting requested configuration because "
+             << "no capabilities were supported.";
+    return false;
   }
-
-  const std::string uma_name_;
-  bool is_request_reported_;
-  bool is_support_reported_;
-};
-
-WebEncryptedMediaClientImpl::WebEncryptedMediaClientImpl(
-    CdmFactory* cdm_factory,
-    MediaPermission* media_permission)
-    : key_systems_(KeySystems::GetInstance()),
-      cdm_factory_(cdm_factory),
-      media_permission_(media_permission),
-      weak_factory_(this) {
-  DCHECK(media_permission);
-}
-
-WebEncryptedMediaClientImpl::~WebEncryptedMediaClientImpl() {
+  // 5. Return media type capabilities.
+  return true;
 }
 
-void WebEncryptedMediaClientImpl::requestMediaKeySystemAccess(
-    blink::WebEncryptedMediaRequest request) {
-  // TODO(jrummell): This should be asynchronous, ideally not on the main
-  // thread.
-
-  // Continued from requestMediaKeySystemAccess(), step 7, from
-  // https://w3c.github.io/encrypted-media/#requestmediakeysystemaccess
-  //
-  // 7.1. If keySystem is not one of the Key Systems supported by the user
-  //      agent, reject promise with with a new DOMException whose name is
-  //      NotSupportedError. String comparison is case-sensitive.
-  if (!base::IsStringASCII(request.keySystem())) {
-    request.requestNotSupported("Only ASCII keySystems are supported");
-    return;
-  }
-
-  // Report this request to the UMA.
-  std::string key_system = base::UTF16ToASCII(request.keySystem());
-  GetReporter(key_system)->ReportRequested();
+bool KeySystemConfigSelector::IsSupportedContentType(
+    const std::string& key_system,
+    EmeMediaType media_type,
+    const std::string& container_mime_type,
+    const std::string& codecs) {
+  // TODO(sandersd): Move contentType parsing from Blink to here so that invalid
+  // parameters can be rejected. http://crbug.com/417561
+  std::string container_lower = base::StringToLowerASCII(container_mime_type);
 
-  if (!key_systems_.IsSupportedKeySystem(key_system)) {
-    request.requestNotSupported("Unsupported keySystem");
-    return;
+  // Check that |container_mime_type| and |codecs| are supported by the CDM.
+  // This check does not handle extended codecs, so extended codec information
+  // is stripped.
+  std::vector<std::string> codec_vector;
+  net::ParseCodecString(codecs, &codec_vector, true);
+  if (!key_systems_->IsSupportedCodecCombination(
+           key_system, media_type, container_lower, codec_vector)) {
+    return false;
   }
 
-  // 7.2-7.4. Implemented by SelectSupportedConfiguration().
-  SelectSupportedConfiguration(request, false, false);
-}
-
-void WebEncryptedMediaClientImpl::SelectSupportedConfiguration(
-    blink::WebEncryptedMediaRequest request,
-    bool was_permission_requested,
-    bool is_permission_granted) {
-  // Continued from requestMediaKeySystemAccess(), step 7.1, from
-  // https://w3c.github.io/encrypted-media/#requestmediakeysystemaccess
-  //
-  // 7.2. Let implementation be the implementation of keySystem.
-  std::string key_system = base::UTF16ToASCII(request.keySystem());
-
-  // 7.3. For each value in supportedConfigurations:
-  const blink::WebVector<blink::WebMediaKeySystemConfiguration>&
-      configurations = request.supportedConfigurations();
-  for (size_t i = 0; i < configurations.size(); i++) {
-    // 7.3.1. Let candidate configuration be the value.
-    const blink::WebMediaKeySystemConfiguration& candidate_configuration =
-        configurations[i];
-    // 7.3.2. Let supported configuration be the result of executing the Get
-    //        Supported Configuration algorithm on implementation, candidate
-    //        configuration, and origin.
-    // 7.3.3. If supported configuration is not null, [initialize and return a
-    //        new MediaKeySystemAccess object.]
-    blink::WebMediaKeySystemConfiguration accumulated_configuration;
-    ConfigurationSupport supported = GetSupportedConfiguration(
-        key_systems_, key_system, candidate_configuration,
-        was_permission_requested, is_permission_granted,
-        &accumulated_configuration);
-    switch (supported) {
-      case CONFIGURATION_NOT_SUPPORTED:
-        continue;
-      case CONFIGURATION_REQUIRES_PERMISSION:
-        if (was_permission_requested) {
-          DVLOG(2) << "Rejecting requested configuration because "
-                   << "permission was denied.";
-          continue;
-        }
-        media_permission_->RequestPermission(
-            MediaPermission::PROTECTED_MEDIA_IDENTIFIER,
-            GURL(request.securityOrigin().toString()),
-            // Try again with |was_permission_requested| true and
-            // |is_permission_granted| the value of the permission.
-            base::Bind(
-                &WebEncryptedMediaClientImpl::SelectSupportedConfiguration,
-                weak_factory_.GetWeakPtr(), request, true));
-        return;
-      case CONFIGURATION_SUPPORTED:
-        // Report that this request succeeded to the UMA.
-        GetReporter(key_system)->ReportSupported();
-        request.requestSucceeded(WebContentDecryptionModuleAccessImpl::Create(
-            request.keySystem(), accumulated_configuration,
-            request.securityOrigin(), weak_factory_.GetWeakPtr()));
-        return;
-    }
+  // Check that |container_mime_type| is supported by Chrome. This can only
+  // happen if the CDM declares support for a container that Chrome does not.
+  if (!net::IsSupportedMediaMimeType(container_lower)) {
+    NOTREACHED();
+    return false;
   }
 
-  // 7.4. Reject promise with a new DOMException whose name is
-  //      NotSupportedError.
-  request.requestNotSupported(
-      "None of the requested configurations were supported.");
-}
-
-void WebEncryptedMediaClientImpl::CreateCdm(
-    const blink::WebString& key_system,
-    bool allow_distinctive_identifier,
-    bool allow_persistent_state,
-    const blink::WebSecurityOrigin& security_origin,
-    blink::WebContentDecryptionModuleResult result) {
-  WebContentDecryptionModuleImpl::Create(
-      cdm_factory_, key_system, allow_distinctive_identifier,
-      allow_persistent_state, security_origin, result);
-}
-
-// Lazily create Reporters.
-WebEncryptedMediaClientImpl::Reporter* WebEncryptedMediaClientImpl::GetReporter(
-    const std::string& key_system) {
-  std::string uma_name = GetKeySystemNameForUMA(key_system);
-  Reporter* reporter = reporters_.get(uma_name);
-  if (reporter != nullptr)
-    return reporter;
-
-  // Reporter not found, so create one.
-  auto result =
-      reporters_.add(uma_name, make_scoped_ptr(new Reporter(uma_name)));
-  DCHECK(result.second);
-  return result.first->second;
+  // Check that |codecs| are supported by Chrome. This is done primarily to
+  // validate extended codecs, but it also ensures that the CDM cannot support
+  // codecs that Chrome does not (which could complicate the robustness
+  // algorithm).
+  if (codec_vector.empty())
+    return true;
+  codec_vector.clear();
+  net::ParseCodecString(codecs, &codec_vector, false);
+  return (net::IsSupportedStrictMediaMimeType(container_lower, codec_vector) ==
+          net::IsSupported);
 }
 
 }  // namespace media