Don't update Media.EME.KeySystemSupport UMA for unprefixed EME

content/renderer/renderer_blink_platform_impl.cc

 // Copyright (c) 2012 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 "content/renderer/renderer_blink_platform_impl.h"
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
@@ skipping 401 matching lines @@
 
     // Chromium only supports ASCII parameters.
     if (!base::IsStringASCII(key_system))
       return IsNotSupported;
 
     std::string key_system_ascii =
         media::GetUnprefixedKeySystemName(base::UTF16ToASCII(key_system));
     std::vector<std::string> strict_codecs;
     net::ParseCodecString(ToASCIIOrEmpty(codecs), &strict_codecs, true);
 
-    if (!media::IsSupportedKeySystemWithMediaMimeType(
+    if (!media::PrefixedIsSupportedKeySystemWithMediaMimeType(
             mime_type_ascii, strict_codecs, key_system_ascii)) {
       return IsNotSupported;
     }
 
     // Continue processing the mime_type and codecs.
   }
 
   // Check list of strict codecs to see if it is supported.
   if (net::IsStrictMediaMimeType(mime_type_ascii)) {
     // Check if the codecs are a perfect match.
@@ skipping 22 matching lines @@
   if (mime_type_ascii.empty())
     return false;
   return media::StreamParserFactory::IsTypeSupported(
       mime_type_ascii, parsed_codec_ids);
 }
 
 bool RendererBlinkPlatformImpl::MimeRegistry::supportsEncryptedMediaMIMEType(
     const WebString& key_system,
     const WebString& mime_type,
     const WebString& codecs) {
+  // TODO(jrummell): This method is only used by unprefixed EME, and should not

[ddorwin, 2015/02/05 00:51:23]

This applies to the whole method (right?), so it should be above 461.

[jrummell, 2015/02/05 01:04:20]

Done.

+  // be called when http://crbug.com/385874 is fixed. Remove this method once
+  // that happens.
+
   // Chromium only supports ASCII parameters.
   if (!base::IsStringASCII(key_system) || !base::IsStringASCII(mime_type) ||
       !base::IsStringASCII(codecs)) {
     return false;
   }
 
   if (key_system.isEmpty())
     return false;
 
   const std::string mime_type_ascii = base::UTF16ToASCII(mime_type);
@@ skipping 749 matching lines @@
 //------------------------------------------------------------------------------
 
 void RendererBlinkPlatformImpl::MockBatteryStatusChangedForTesting(
     const blink::WebBatteryStatus& status) {
   if (!g_test_battery_status_listener)
     return;
   g_test_battery_status_listener->updateBatteryStatus(status);
 }
 
 }  // namespace content