[eme] Ignore capabilities with unrecognized parameters

third_party/WebKit/Source/modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.cpp

 // 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.
 
 #include "modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.h"
 
 #include <algorithm>
 
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
@@ skipping 35 matching lines @@
 }
 
 static WebVector<WebMediaKeySystemMediaCapability> convertCapabilities(
     const HeapVector<MediaKeySystemMediaCapability>& capabilities) {
   WebVector<WebMediaKeySystemMediaCapability> result(capabilities.size());
   for (size_t i = 0; i < capabilities.size(); ++i) {
     const WebString& contentType = capabilities[i].contentType();
     result[i].contentType = contentType;
     ParsedContentType type(contentType);
     if (type.isValid()) {
-      // FIXME: Fail if there are unrecognized parameters.
-      // http://crbug.com/690131
+      // From
+      // http://w3c.github.io/encrypted-media/#get-supported-capabilities-for-audio-video-type
+      // "If the user agent does not recognize one or more parameters,
+      // continue to the next iteration." There is no way to enumerate the
+      // parameters, so only look up "codecs" if a single parameter is
+      // present. Chromium expects "codecs" to be provided, so this capability
+      // will be skipped if codecs is not the only parameter specified.
       result[i].mimeType = type.mimeType();
-      result[i].codecs = type.parameterValueForName("codecs");
+      if (type.parameterCount() == 1u)
+        result[i].codecs = type.parameterValueForName("codecs");
     }
     result[i].robustness = capabilities[i].robustness();
   }
   return result;
 }
 
 static WebMediaKeySystemConfiguration::Requirement convertMediaKeysRequirement(
     const String& requirement) {
   if (requirement == "required")
     return WebMediaKeySystemConfiguration::Requirement::Required;
@@ skipping 255 matching lines @@
   WebEncryptedMediaClient* mediaClient =
       controller->encryptedMediaClient(executionContext);
   mediaClient->requestMediaKeySystemAccess(
       WebEncryptedMediaRequest(initializer));
 
   // 7. Return promise.
   return promise;
 }
 
 }  // namespace blink