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

Unified Diff: ui/gl/gl_image_memory.cc

Issue 1282313002: Add YUV_420_BIPLANAR to gfx::BufferFormat. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@gmb-planes
Patch Set: Enable it on Mac Created 5 years, 4 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
Index: ui/gl/gl_image_memory.cc
diff --git a/ui/gl/gl_image_memory.cc b/ui/gl/gl_image_memory.cc
index f34611851c042ab8a3fdb92ae4df2ce69c882744..0409d6dd704538273b20edc77a12558791f30014 100644
--- a/ui/gl/gl_image_memory.cc
+++ b/ui/gl/gl_image_memory.cc
@@ -47,6 +47,7 @@ bool ValidFormat(BufferFormat format) {
return true;
case BufferFormat::RGBX_8888:
case BufferFormat::YUV_420:
+ case BufferFormat::YUV_420_BIPLANAR:
return false;
}
@@ -62,6 +63,7 @@ bool IsCompressedFormat(BufferFormat format) {
case BufferFormat::DXT5:
case BufferFormat::ETC1:
case BufferFormat::YUV_420:
+ case BufferFormat::YUV_420_BIPLANAR:
reveman 2015/08/10 22:40:55 hm, I don't think we should return true here for t
Andre 2015/08/11 03:50:46 Done.
return true;
case BufferFormat::R_8:
case BufferFormat::RGBA_4444:
@@ -96,6 +98,7 @@ GLenum TextureFormat(BufferFormat format) {
return GL_BGRA_EXT;
case BufferFormat::RGBX_8888:
case BufferFormat::YUV_420:
+ case BufferFormat::YUV_420_BIPLANAR:
NOTREACHED();
return 0;
}
@@ -123,6 +126,7 @@ GLenum DataType(BufferFormat format) {
case BufferFormat::ETC1:
case BufferFormat::RGBX_8888:
case BufferFormat::YUV_420:
+ case BufferFormat::YUV_420_BIPLANAR:
NOTREACHED();
return 0;
}
@@ -203,6 +207,7 @@ bool GLImageMemory::StrideInBytes(size_t width,
return true;
case BufferFormat::RGBX_8888:
case BufferFormat::YUV_420:
+ case BufferFormat::YUV_420_BIPLANAR:
NOTREACHED();
return false;
}

Powered by Google App Engine
This is Rietveld 408576698