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

Side by Side Diff: content/browser/media/media_browsertest.cc

Issue 263723004: Revert of Add correct support for videos with YUVJ420P color format, in the software conversion path. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@yuvnopic
Patch Set: Created 6 years, 7 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 unified diff | Download patch
« no previous file with comments | « no previous file | media/base/simd/convert_yuv_to_rgb.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/media/media_browsertest.h" 5 #include "content/browser/media/media_browsertest.h"
6 6
7 #include "base/strings/stringprintf.h" 7 #include "base/strings/stringprintf.h"
8 #include "base/strings/utf_string_conversions.h" 8 #include "base/strings/utf_string_conversions.h"
9 #include "content/public/browser/web_contents.h" 9 #include "content/public/browser/web_contents.h"
10 #include "content/public/common/url_constants.h" 10 #include "content/public/common/url_constants.h"
(...skipping 213 matching lines...) Expand 10 before | Expand all | Expand 10 after
224 IN_PROC_BROWSER_TEST_F(MediaTest, MAYBE(Yuv420pVp8)) { 224 IN_PROC_BROWSER_TEST_F(MediaTest, MAYBE(Yuv420pVp8)) {
225 RunColorFormatTest("yuv420p.webm", "ENDED"); 225 RunColorFormatTest("yuv420p.webm", "ENDED");
226 } 226 }
227 227
228 #if defined(USE_PROPRIETARY_CODECS) 228 #if defined(USE_PROPRIETARY_CODECS)
229 IN_PROC_BROWSER_TEST_F(MediaTest, MAYBE(Yuv420pH264)) { 229 IN_PROC_BROWSER_TEST_F(MediaTest, MAYBE(Yuv420pH264)) {
230 RunColorFormatTest("yuv420p.mp4", "ENDED"); 230 RunColorFormatTest("yuv420p.mp4", "ENDED");
231 } 231 }
232 232
233 IN_PROC_BROWSER_TEST_F(MediaTest, MAYBE(Yuvj420pH264)) { 233 IN_PROC_BROWSER_TEST_F(MediaTest, MAYBE(Yuvj420pH264)) {
234 RunColorFormatTest("yuvj420p.mp4", "ENDED"); 234 // TODO(rileya): Support YUVJ420P properly http://crbug.com/310273
235 RunColorFormatTest("yuvj420p.mp4", "FAILED");
235 } 236 }
236 237
237 IN_PROC_BROWSER_TEST_F(MediaTest, MAYBE(Yuv422pH264)) { 238 IN_PROC_BROWSER_TEST_F(MediaTest, MAYBE(Yuv422pH264)) {
238 RunColorFormatTest("yuv422p.mp4", "ENDED"); 239 RunColorFormatTest("yuv422p.mp4", "ENDED");
239 } 240 }
240 241
241 IN_PROC_BROWSER_TEST_F(MediaTest, MAYBE(Yuv444pH264)) { 242 IN_PROC_BROWSER_TEST_F(MediaTest, MAYBE(Yuv444pH264)) {
242 // TODO(scherkus): Support YUV444 http://crbug.com/104711 243 // TODO(scherkus): Support YUV444 http://crbug.com/104711
243 RunColorFormatTest("yuv444p.mp4", "ERROR"); 244 RunColorFormatTest("yuv444p.mp4", "ERROR");
244 } 245 }
245 246
246 #if defined(OS_CHROMEOS) 247 #if defined(OS_CHROMEOS)
247 IN_PROC_BROWSER_TEST_F(MediaTest, Yuv420pMpeg4) { 248 IN_PROC_BROWSER_TEST_F(MediaTest, Yuv420pMpeg4) {
248 RunColorFormatTest("yuv420p.avi", "ENDED"); 249 RunColorFormatTest("yuv420p.avi", "ENDED");
249 } 250 }
250 #endif // defined(OS_CHROMEOS) 251 #endif // defined(OS_CHROMEOS)
251 #endif // defined(USE_PROPRIETARY_CODECS) 252 #endif // defined(USE_PROPRIETARY_CODECS)
252 253
253 } // namespace content 254 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | media/base/simd/convert_yuv_to_rgb.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698