Bug 1370976 - Add webm MediaMIMEType gtest. r?gerald draft
authorRalph Giles <giles@mozilla.com>
Wed, 31 May 2017 13:35:15 -0700
changeset 591220 8930f6057835c148465405920090282b2065a838
parent 591094 e61060be36424240058f8bef4c5597f401bc8b7e
child 632472 36f29ab36056f1408ed6294352d2aa5b3262bccb
push id63007
push userbmo:giles@thaumas.net
push dateThu, 08 Jun 2017 21:17:44 +0000
reviewersgerald
bugs1370976
milestone55.0a1
Bug 1370976 - Add webm MediaMIMEType gtest. r?gerald Make sure we test the other major container we need to parse correctly. MozReview-Commit-ID: AnrGADFXPkw
dom/media/gtest/TestMediaMIMETypes.cpp
--- a/dom/media/gtest/TestMediaMIMETypes.cpp
+++ b/dom/media/gtest/TestMediaMIMETypes.cpp
@@ -179,17 +179,19 @@ TEST(MediaMIMETypes, MediaExtendedMIMETy
     bool mEqualsLiteralVideoSlashMp4; // tests `== "video/mp4"`
     bool mHaveCodecs;
   } tests[] =
   { // in                    Type().AsString  app    audio  video ==v/mp4 codecs
     { "video/mp4",           "video/mp4",     false, false, true,  true,  false },
     { "video/mp4; codecs=0", "video/mp4",     false, false, true,  true,  true  },
     { "VIDEO/MP4",           "video/mp4",     false, false, true,  true,  false },
     { "audio/mp4",           "audio/mp4",     false, true,  false, false, false },
-    { "application/x",       "application/x", true, false,  false, false, false }
+    { "video/webm",          "video/webm",    false, false, true,  false, false },
+    { "audio/webm",          "audio/webm",    false, true,  false, false, false },
+    { "application/x",       "application/x", true,  false, false, false, false }
   };
 
   for (const auto& test : tests) {
     Maybe<MediaExtendedMIMEType> type = MakeMediaExtendedMIMEType(test.mTypeString);
     EXPECT_TRUE(type.isSome())
       << "MakeMediaExtendedMIMEType(\"" << test.mTypeString << "\").isSome()";
     EXPECT_TRUE(type->OriginalString().EqualsASCII(test.mTypeString))
       << "MakeMediaExtendedMIMEType(\"" << test.mTypeString << "\")->AsString() == \"" << test.mTypeAsString << "\"";