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

Unified Diff: media/base/video_frame_unittest.cc

Issue 10024072: Cull unnecessary media::VideoFrame::Formats from the enum. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: shader_bench builds after having RGBA experimental code ripped out. Created 8 years, 8 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 | « media/base/video_frame.cc ('k') | media/media.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/video_frame_unittest.cc
diff --git a/media/base/video_frame_unittest.cc b/media/base/video_frame_unittest.cc
index 9548797e696b7ad3461d97aa340f69e9791316bc..3357ed215b5f46c25f516e9bf2264a2d6eb12507 100644
--- a/media/base/video_frame_unittest.cc
+++ b/media/base/video_frame_unittest.cc
@@ -47,7 +47,7 @@ void ExpectFrameColor(media::VideoFrame* yv12_frame, uint32 expect_rgb_color) {
yv12_frame->stride(VideoFrame::kVPlane));
scoped_refptr<media::VideoFrame> rgb_frame;
- rgb_frame = media::VideoFrame::CreateFrame(VideoFrame::RGBA,
+ rgb_frame = media::VideoFrame::CreateFrame(VideoFrame::RGB32,
yv12_frame->width(),
yv12_frame->height(),
yv12_frame->GetTimestamp(),
@@ -202,16 +202,8 @@ TEST(VideoFrame, CheckFrameExtents) {
// and the expected hash of all planes if filled with kFillByte (defined in
// ExpectFrameExtents).
ExpectFrameExtents(
- VideoFrame::RGB555, 1, 2, "31f7739efc76b5d9cb51361ba82533fa");
- ExpectFrameExtents(
- VideoFrame::RGB565, 1, 2, "31f7739efc76b5d9cb51361ba82533fa");
- ExpectFrameExtents(
- VideoFrame::RGB24, 1, 3, "84361ae9d4b6d4641a11474b3a7a2260");
- ExpectFrameExtents(
VideoFrame::RGB32, 1, 4, "de6d3d567e282f6a38d478f04fc81fb0");
ExpectFrameExtents(
- VideoFrame::RGBA, 1, 4, "de6d3d567e282f6a38d478f04fc81fb0");
- ExpectFrameExtents(
VideoFrame::YV12, 3, 1, "71113bdfd4c0de6cf62f48fb74f7a0b1");
ExpectFrameExtents(
VideoFrame::YV16, 3, 1, "9bb99ac3ff350644ebff4d28dc01b461");
« no previous file with comments | « media/base/video_frame.cc ('k') | media/media.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698