Fix: Adding list of supported codecs for MP4 containers

net/base/mime_util.h

Index: net/base/mime_util.h
diff --git a/net/base/mime_util.h b/net/base/mime_util.h
index 9662e9656e397de2d48adedbfae590a3249ff4a7..764bd8ecac2ab330d47f22c61e8c1781b3bd9185 100644
--- a/net/base/mime_util.h
+++ b/net/base/mime_util.h
@@ -76,12 +76,14 @@ NET_EXPORT void ParseCodecString(const std::string& codecs,
 // certain subset of codecs.
 NET_EXPORT bool IsStrictMediaMimeType(const std::string& mime_type);
 
+enum CanPlayType { IsNotSupported, IsSupported, MayBeSupported };
+
 // Check to see if a particular MIME type is in our list which only supports a
 // certain subset of codecs. Returns true if and only if all codecs are
 // supported for that specific MIME type, false otherwise. If this returns
 // false you will still need to check if the media MIME tpyes and codecs are
 // supported.

[Ryan Sleevi, 2014/05/15 22:48:35]

This comment block is now out of date and needs to be updated.

This "smells" like a layering violation though - letting details about Blink
seep into net/.

[amogh.bihani, 2014/05/16 11:25:29]

done.
That's why I had made a new method for checking MP4 and HLS mime types. Acolwell
had advised to change this to return enum. This does look cleaner than my
previous approach.

-NET_EXPORT bool IsSupportedStrictMediaMimeType(
+NET_EXPORT CanPlayType IsSupportedStrictMediaMimeType(
     const std::string& mime_type,
     const std::vector<std::string>& codecs);