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..98307f965a7b3ffe3399ff1dea90f47738f09f2d 100644
--- a/content/renderer/renderer_webkitplatformsupport_impl.cc
+++ b/content/renderer/renderer_webkitplatformsupport_impl.cc
@@ -438,6 +438,10 @@ RendererWebKitPlatformSupportImpl::MimeRegistry::supportsMediaMIMEType(
     if (net::IsSupportedStrictMediaMimeType(mime_type_ascii, strict_codecs))
       return IsSupported;
 
+    // H.264 codecs have profile version in their names.

[ddorwin, 2014/04/29 17:12:54]

H.264 does not match the function name.

+    if (net::IsSupportedStrictMP4MediaMimeType(mime_type_ascii, strict_codecs))

[ddorwin, 2014/04/29 17:12:54]

You might also comment that since we do not check for exact profile versions, we
will only return maybe.

However, it would really be better to have the function make that determination.
(The current implementation would always return maybe, but this code shouldn't
need to know that.) Then you wouldn't need the comment.

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