Fix: Adding list of supported codecs for MP4 containers

net/base/mime_util_unittest.cc

Index: net/base/mime_util_unittest.cc
diff --git a/net/base/mime_util_unittest.cc b/net/base/mime_util_unittest.cc
index 61719889034a7ea3c7610f836063b082d23c16e7..ef2d9d725d6334855939fa1072d2d3988d5ce39f 100644
--- a/net/base/mime_util_unittest.cc
+++ b/net/base/mime_util_unittest.cc
@@ -113,15 +113,13 @@ TEST(MimeUtilTest, StrictMediaMimeType) {
   EXPECT_TRUE(IsStrictMediaMimeType("audio/mp3"));
   EXPECT_TRUE(IsStrictMediaMimeType("audio/x-mp3"));
 
-  // TODO(amogh.bihani): These will be fixed http://crbug.com/53193
-  EXPECT_FALSE(IsStrictMediaMimeType("video/mp4"));
-  EXPECT_FALSE(IsStrictMediaMimeType("video/x-m4v"));
-  EXPECT_FALSE(IsStrictMediaMimeType("audio/mp4"));
-  EXPECT_FALSE(IsStrictMediaMimeType("audio/x-m4a"));
-
-  EXPECT_FALSE(IsStrictMediaMimeType("application/x-mpegurl"));
-  EXPECT_FALSE(IsStrictMediaMimeType("application/vnd.apple.mpegurl"));
-  // ---------------------------------------------------------------------------
+  EXPECT_TRUE(IsStrictMediaMimeType("video/mp4"));
+  EXPECT_TRUE(IsStrictMediaMimeType("video/x-m4v"));
+  EXPECT_TRUE(IsStrictMediaMimeType("audio/mp4"));
+  EXPECT_TRUE(IsStrictMediaMimeType("audio/x-m4a"));
+
+  EXPECT_TRUE(IsStrictMediaMimeType("application/x-mpegurl"));

[acolwell GONE FROM CHROMIUM, 2014/05/22 20:21:42]

I would expect these to be only true for Android. We don't support HLS on
desktop Chrome.

[amogh.bihani, 2014/05/23 10:57:29]

Yes HLS is supported only on Android, but that would be true from the complete
browser's perspective. In browser we first check whether the mime_type is
supported or not[1] and then move forward. This method comes after the check,
hence we do not put #ifdef in its list. Since all the mime types are available
in the list, this unittest is supposed to pass on all the platforms. (You may
notice that in this test, video/ogg is passing, even for android, because of
this reason.)
However, the media_canplaytype_browsertest should fail (and it does). In this
all the HLS tests return "" for canPlayType() query for non android platforms
(line 765-768 in that file of this patchset).

-------
[1]
https://code.google.com/p/chromium/codesearch#chromium/src/content/renderer/r...

+  EXPECT_TRUE(IsStrictMediaMimeType("application/vnd.apple.mpegurl"));
 
   EXPECT_FALSE(IsStrictMediaMimeType("video/unknown"));
   EXPECT_FALSE(IsStrictMediaMimeType("audio/unknown"));