diff options
| author | dcastagna <dcastagna@chromium.org> | 2016-02-10 17:30:05 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-02-11 01:31:07 +0000 |
| commit | dcaccb9c51299dfdf2cd925fd1a3ea6c480862a7 (patch) | |
| tree | 7198b5912727c72024fafcf2244b615352102218 /media/video | |
| parent | b72ed077b4cb7309c5e3ee6bb80944a3deab6250 (diff) | |
| download | chromium_src-dcaccb9c51299dfdf2cd925fd1a3ea6c480862a7.zip chromium_src-dcaccb9c51299dfdf2cd925fd1a3ea6c480862a7.tar.gz chromium_src-dcaccb9c51299dfdf2cd925fd1a3ea6c480862a7.tar.bz2 | |
Enable NV12 VideoFrames on Mac.
As described in crbug.com/524582, we observed a reduction in power
consumption when this format is used to display videos directly using
the WindowServer.
The downside of this format is that a conversion from YUV to RGB will
be performed on the GPU using GL when the VideoFrame has to be used
from GL.
BUG=524582
Review URL: https://codereview.chromium.org/1686443002
Cr-Commit-Position: refs/heads/master@{#374825}
Diffstat (limited to 'media/video')
| -rw-r--r-- | media/video/gpu_memory_buffer_video_frame_pool.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/media/video/gpu_memory_buffer_video_frame_pool.cc b/media/video/gpu_memory_buffer_video_frame_pool.cc index 7ac0cd7..e7ffd99 100644 --- a/media/video/gpu_memory_buffer_video_frame_pool.cc +++ b/media/video/gpu_memory_buffer_video_frame_pool.cc @@ -202,8 +202,7 @@ unsigned ImageInternalFormat(VideoPixelFormat format, size_t plane) { return GL_RED_EXT; case PIXEL_FORMAT_NV12: DCHECK_LE(plane, 1u); - DLOG(WARNING) << "NV12 format not supported yet"; - return 0; // TODO(andresantoso): Implement extension for NV12. + return GL_RGB_YCBCR_420V_CHROMIUM; case PIXEL_FORMAT_UYVY: DCHECK_EQ(0u, plane); return GL_RGB_YCBCR_422_CHROMIUM; |