Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(852)

Unified Diff: media/base/mime_util_unittest.cc

Issue 1624703002: Implement support for vp9 in ISO-BMFF (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/mime_util_unittest.cc
diff --git a/media/base/mime_util_unittest.cc b/media/base/mime_util_unittest.cc
index ca1f393723567ca8a4dc7fdd08db2b4c05892525..bc9de51eec578e856902be9728529e3345744dc0 100644
--- a/media/base/mime_util_unittest.cc
+++ b/media/base/mime_util_unittest.cc
@@ -336,7 +336,8 @@ TEST(IsCodecSupportedOnPlatformTest, ClearCodecBehaviorWithAndroidPipeline) {
break;
case MimeUtil::VP9:
- EXPECT_EQ(info.supports_vp9, result);
+ // MediaPlayer only supports VP9 in WebM.
+ EXPECT_FALSE(result);
break;
case MimeUtil::HEVC_MAIN:
@@ -344,6 +345,16 @@ TEST(IsCodecSupportedOnPlatformTest, ClearCodecBehaviorWithAndroidPipeline) {
break;
}
});
+
+ // Verify vp9 support in WebM.
+ RunCodecSupportTest(
+ states_to_vary, test_states,
+ [](const MimeUtil::PlatformInfo& info, MimeUtil::Codec codec) {
+ const bool result = MimeUtil::IsCodecSupportedOnPlatform(
+ codec, "video/webm", true, info);
+ if (codec == MimeUtil::VP9)
+ EXPECT_EQ(info.supports_vp9, result);
+ });
}
TEST(IsCodecSupportedOnPlatformTest, ClearCodecBehaviorWithUnifiedPipeline) {

Powered by Google App Engine
This is Rietveld 408576698