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

content/renderer/renderer_blink_platform_impl.cc

Index: content/renderer/renderer_blink_platform_impl.cc
diff --git a/content/renderer/renderer_blink_platform_impl.cc b/content/renderer/renderer_blink_platform_impl.cc
index 2894dc6cefbf4328f28d36fd75d0bc67b495fa52..9d81ad1f83a090371255aa1362816171988e96e6 100644
--- a/content/renderer/renderer_blink_platform_impl.cc
+++ b/content/renderer/renderer_blink_platform_impl.cc
@@ -420,7 +420,7 @@ RendererBlinkPlatformImpl::MimeRegistry::supportsMediaMIMEType(
     net::ParseCodecString(ToASCIIOrEmpty(codecs), &strict_codecs, true);
 
     if (!media::IsSupportedKeySystemWithMediaMimeType(

[ddorwin, 2015/01/27 00:51:51]

Instead of adding a bool, the purpose of which is not obvious at the call site,
change the function name to contain "Prefixed". You can hide the bool internally
if necessary.

(I'm not sure where to put "Prefixed". Prefixing the function name would be the
most obvious, but is probably not style compliant. Still, as far as readability
is concerned, I think it's best.

[jrummell, 2015/02/03 00:40:49]

Done.

-            mime_type_ascii, strict_codecs, key_system_ascii)) {
+            mime_type_ascii, strict_codecs, key_system_ascii, true)) {
       return IsNotSupported;
     }
 
@@ -479,7 +479,7 @@ bool RendererBlinkPlatformImpl::MimeRegistry::supportsEncryptedMediaMIMEType(
                         strip_suffix);
 
   return media::IsSupportedKeySystemWithMediaMimeType(
-      mime_type_ascii, codec_vector, base::UTF16ToASCII(key_system));
+      mime_type_ascii, codec_vector, base::UTF16ToASCII(key_system), true);

[ddorwin, 2015/01/27 00:51:51]

That makes true incorrect.

[jrummell, 2015/02/03 00:40:49]

Done.

 }
 
 WebString RendererBlinkPlatformImpl::MimeRegistry::mimeTypeForExtension(