Commit c464fed9 authored by Hirokazu Honda's avatar Hirokazu Honda Committed by Commit Bot

media/base: Fix VideoFrame::BytesElement() to P016LE

Bug: None
Test: None
Change-Id: I252ce0b7eb284ff1ca34a47191dd12f0a81ddcc6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2350720
Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: default avatarDale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#797195}
parent 01a52c5d
...@@ -875,7 +875,6 @@ int VideoFrame::BytesPerElement(VideoPixelFormat format, size_t plane) { ...@@ -875,7 +875,6 @@ int VideoFrame::BytesPerElement(VideoPixelFormat format, size_t plane) {
case PIXEL_FORMAT_YUV420P12: case PIXEL_FORMAT_YUV420P12:
case PIXEL_FORMAT_YUV422P12: case PIXEL_FORMAT_YUV422P12:
case PIXEL_FORMAT_YUV444P12: case PIXEL_FORMAT_YUV444P12:
case PIXEL_FORMAT_P016LE:
return 2; return 2;
case PIXEL_FORMAT_NV12: case PIXEL_FORMAT_NV12:
case PIXEL_FORMAT_NV21: { case PIXEL_FORMAT_NV21: {
...@@ -883,6 +882,11 @@ int VideoFrame::BytesPerElement(VideoPixelFormat format, size_t plane) { ...@@ -883,6 +882,11 @@ int VideoFrame::BytesPerElement(VideoPixelFormat format, size_t plane) {
DCHECK_LT(plane, base::size(bytes_per_element)); DCHECK_LT(plane, base::size(bytes_per_element));
return bytes_per_element[plane]; return bytes_per_element[plane];
} }
case PIXEL_FORMAT_P016LE: {
static const int bytes_per_element[] = {1, 2};
DCHECK_LT(plane, base::size(bytes_per_element));
return bytes_per_element[plane] * 2;
}
case PIXEL_FORMAT_YV12: case PIXEL_FORMAT_YV12:
case PIXEL_FORMAT_I420: case PIXEL_FORMAT_I420:
case PIXEL_FORMAT_I422: case PIXEL_FORMAT_I422:
......
...@@ -686,6 +686,7 @@ TEST(VideoFrame, AllocationSize_OddSize) { ...@@ -686,6 +686,7 @@ TEST(VideoFrame, AllocationSize_OddSize) {
case PIXEL_FORMAT_YUV420P9: case PIXEL_FORMAT_YUV420P9:
case PIXEL_FORMAT_YUV420P10: case PIXEL_FORMAT_YUV420P10:
case PIXEL_FORMAT_YUV420P12: case PIXEL_FORMAT_YUV420P12:
case PIXEL_FORMAT_P016LE:
EXPECT_EQ(72u, VideoFrame::AllocationSize(format, size)) EXPECT_EQ(72u, VideoFrame::AllocationSize(format, size))
<< VideoPixelFormatToString(format); << VideoPixelFormatToString(format);
break; break;
...@@ -708,7 +709,6 @@ TEST(VideoFrame, AllocationSize_OddSize) { ...@@ -708,7 +709,6 @@ TEST(VideoFrame, AllocationSize_OddSize) {
case PIXEL_FORMAT_I420A: case PIXEL_FORMAT_I420A:
case PIXEL_FORMAT_ABGR: case PIXEL_FORMAT_ABGR:
case PIXEL_FORMAT_XBGR: case PIXEL_FORMAT_XBGR:
case PIXEL_FORMAT_P016LE:
case PIXEL_FORMAT_XR30: case PIXEL_FORMAT_XR30:
case PIXEL_FORMAT_XB30: case PIXEL_FORMAT_XB30:
EXPECT_EQ(60u, VideoFrame::AllocationSize(format, size)) EXPECT_EQ(60u, VideoFrame::AllocationSize(format, size))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment