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

Unified Diff: media/base/video_frame_unittest.cc

Issue 2480203002: ui: Cleanup class/struct forward declarations (Closed)
Patch Set: Sync CL to position 430550 Created 4 years, 1 month 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/blink/renderer_media_player_interface.h » ('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 730496abc1fe61cdd1a4357db15e2b9564ea342d..865db823aa6e2498aa80a60289e655950720852f 100644
--- a/media/base/video_frame_unittest.cc
+++ b/media/base/video_frame_unittest.cc
@@ -429,6 +429,62 @@ TEST(VideoFrame, CreateFrame_OddWidth) {
EXPECT_EQ(677, frame->coded_size().width());
}
+TEST(VideoFrame, AllocationSize_OddSize) {
+ const gfx::Size size(3, 5);
+ for (unsigned int i = 1u; i <= PIXEL_FORMAT_MAX; ++i) {
+ const VideoPixelFormat format = static_cast<VideoPixelFormat>(i);
+ const size_t allocation_size = VideoFrame::AllocationSize(format, size);
+ switch (format) {
+ case PIXEL_FORMAT_YUV444P9:
+ case PIXEL_FORMAT_YUV444P10:
+ case PIXEL_FORMAT_YUV444P12:
+ EXPECT_EQ(144u, allocation_size) << VideoPixelFormatToString(format);
+ break;
+ case PIXEL_FORMAT_YUV422P9:
+ case PIXEL_FORMAT_YUV422P10:
+ case PIXEL_FORMAT_YUV422P12:
+ EXPECT_EQ(96u, allocation_size) << VideoPixelFormatToString(format);
+ break;
+ case PIXEL_FORMAT_YV24:
+ case PIXEL_FORMAT_YUV420P9:
+ case PIXEL_FORMAT_YUV420P10:
+ case PIXEL_FORMAT_YUV420P12:
+ EXPECT_EQ(72u, allocation_size) << VideoPixelFormatToString(format);
+ break;
+ case PIXEL_FORMAT_UYVY:
+ case PIXEL_FORMAT_YUY2:
+ case PIXEL_FORMAT_YV16:
+ EXPECT_EQ(48u, allocation_size) << VideoPixelFormatToString(format);
+ break;
+ case PIXEL_FORMAT_YV12:
+ case PIXEL_FORMAT_I420:
+ case PIXEL_FORMAT_NV12:
+ case PIXEL_FORMAT_NV21:
+ case PIXEL_FORMAT_MT21:
+ EXPECT_EQ(36u, allocation_size) << VideoPixelFormatToString(format);
+ break;
+ case PIXEL_FORMAT_ARGB:
+ case PIXEL_FORMAT_XRGB:
+ case PIXEL_FORMAT_YV12A:
+ case PIXEL_FORMAT_RGB32:
+ EXPECT_EQ(60u, allocation_size) << VideoPixelFormatToString(format);
+ break;
+ case PIXEL_FORMAT_RGB24:
+ EXPECT_EQ(45u, allocation_size) << VideoPixelFormatToString(format);
+ break;
+ case PIXEL_FORMAT_Y16:
+ EXPECT_EQ(30u, allocation_size) << VideoPixelFormatToString(format);
+ break;
+ case PIXEL_FORMAT_Y8:
+ EXPECT_EQ(15u, allocation_size) << VideoPixelFormatToString(format);
+ break;
+ case PIXEL_FORMAT_MJPEG:
+ case PIXEL_FORMAT_UNKNOWN:
+ break;
+ }
+ }
+}
+
TEST(VideoFrameMetadata, SetAndThenGetAllKeysForAllTypes) {
VideoFrameMetadata metadata;
« no previous file with comments | « media/base/video_frame.cc ('k') | media/blink/renderer_media_player_interface.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698