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

Unified Diff: content/browser/media/media_color_browsertest.cc

Issue 2345073002: Clean up the media color format content_browsertests (Closed)
Patch Set: Disable on chromos Created 4 years, 3 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
« no previous file with comments | « content/browser/media/media_browsertest.cc ('k') | content/test/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/media/media_color_browsertest.cc
diff --git a/content/browser/media/media_color_browsertest.cc b/content/browser/media/media_color_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..33f0b79c0ad7e397ab0d46e0bf50e16fb57cb60a
--- /dev/null
+++ b/content/browser/media/media_color_browsertest.cc
@@ -0,0 +1,88 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "build/build_config.h"
+#include "content/browser/media/media_browsertest.h"
+#include "content/public/test/browser_test_utils.h"
+#include "media/base/test_data_util.h"
+
+namespace content {
+
+class MediaColorTest : public MediaBrowserTest {
+ public:
+ void RunColorTest(const std::string& video_file) {
+ base::FilePath path = media::GetTestDataFilePath("blackwhite.html");
+ std::string final_title =
+ RunTest(GetFileUrlWithQuery(path, video_file), kEnded);
+ EXPECT_EQ(kEnded, final_title);
+ }
+};
+
+IN_PROC_BROWSER_TEST_F(MediaColorTest, Yuv420pTheora) {
+ RunColorTest("yuv420p.ogv");
+}
+
+IN_PROC_BROWSER_TEST_F(MediaColorTest, Yuv422pTheora) {
+ RunColorTest("yuv422p.ogv");
+}
+
+IN_PROC_BROWSER_TEST_F(MediaColorTest, Yuv444pTheora) {
+ RunColorTest("yuv444p.ogv");
+}
+
+IN_PROC_BROWSER_TEST_F(MediaColorTest, Yuv420pVp8) {
+ RunColorTest("yuv420p.webm");
+}
+
+IN_PROC_BROWSER_TEST_F(MediaColorTest, Yuv444pVp9) {
+ RunColorTest("yuv444p.webm");
+}
+
+#if defined(USE_PROPRIETARY_CODECS)
+IN_PROC_BROWSER_TEST_F(MediaColorTest, Yuv420pH264) {
+ RunColorTest("yuv420p.mp4");
+}
+
+// This test fails on Android http://crbug.com/647818, and OSX
+// http://crbug.com/647838.
+#if defined(OS_MACOSX) || defined(OS_ANDROID)
+#define MAYBE_Yuvj420pH264 DISABLED_Yuvj420pH264
+#else
+#define MAYBE_Yuvj420pH264 Yuvj420pH264
+#endif
+IN_PROC_BROWSER_TEST_F(MediaColorTest, MAYBE_Yuvj420pH264) {
+ RunColorTest("yuvj420p.mp4");
+}
+
+// This fails on Linux & ChromeOS http://crbug.com/647400, and Windows
+// http://crbug.com/647842.
+#if defined(OS_LINUX) || defined(OS_CHROMEOS) || defined(OS_WIN)
+#define MAYBE_Yuv420pRec709H264 DISABLED_Yuv420pRec709H264
+#else
+#define MAYBE_Yuv420pRec709H264 Yuv420pRec709H264
+#endif
+IN_PROC_BROWSER_TEST_F(MediaColorTest, MAYBE_Yuv420pRec709H264) {
+ RunColorTest("yuv420p_rec709.mp4");
+}
+
+IN_PROC_BROWSER_TEST_F(MediaColorTest, Yuv420pHighBitDepth) {
+ RunColorTest("yuv420p_hi10p.mp4");
+}
+
+IN_PROC_BROWSER_TEST_F(MediaColorTest, Yuv422pH264) {
+ RunColorTest("yuv422p.mp4");
+}
+
+IN_PROC_BROWSER_TEST_F(MediaColorTest, Yuv444pH264) {
+ RunColorTest("yuv444p.mp4");
+}
+
+#if defined(OS_CHROMEOS)
+IN_PROC_BROWSER_TEST_F(MediaColorTest, Yuv420pMpeg4) {
+ RunColorTest("yuv420p.avi");
+}
+#endif // defined(OS_CHROMEOS)
+#endif // defined(USE_PROPRIETARY_CODECS)
+
+} // namespace content
« no previous file with comments | « content/browser/media/media_browsertest.cc ('k') | content/test/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698