Fix: Adding list of supported codecs for MP4 containers

content/renderer/renderer_webkitplatformsupport_impl.cc

Index: content/renderer/renderer_webkitplatformsupport_impl.cc
diff --git a/content/renderer/renderer_webkitplatformsupport_impl.cc b/content/renderer/renderer_webkitplatformsupport_impl.cc
index aa46b53274bfdfdc3fef003131d785c03078ebe1..61c456b7992d5f7a4c7a8c9c80d7caeecd3f5c1e 100644
--- a/content/renderer/renderer_webkitplatformsupport_impl.cc
+++ b/content/renderer/renderer_webkitplatformsupport_impl.cc
@@ -438,6 +438,14 @@ RendererWebKitPlatformSupportImpl::MimeRegistry::supportsMediaMIMEType(
     if (net::IsSupportedStrictMediaMimeType(mime_type_ascii, strict_codecs))
       return IsSupported;
 
+    // Codecs required by MP4 and HLS mime types have a profile paramter in them
+    // Since we do not check the exact profile version supported by the system,
+    // we only return a "maybe" if the codecs are RFC6381 compliant.
+    if (net::IsSupportedStrictMP4OrHLSMediaMimeType(mime_type_ascii,

[acolwell GONE FROM CHROMIUM, 2014/05/12 14:54:18]

I don't think MP4OrHLS should be in the method name. It seems to me that
IsSupportedStrictMediaMimeType() should be changed to return an enum that
includes a MayBe or this method is be something like
MayBeSupportedStrictMediaMimeType() since really you are just trying to capture
the MayBe situations. My preference is the former solution since I don't really
see the point of having 2 methods here.

[amogh.bihani, 2014/05/15 09:05:53]

Done.

+                                                    strict_codecs)) {
+      return MayBeSupported;
+    }
+
     // We support the container, but no codecs were specified.
     if (codecs.isNull())
       return MayBeSupported;