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

Unified Diff: ui/gfx/linux/client_native_pixmap_factory_dmabuf.cc

Issue 2678343011: chromeos: decode video into NV12 format instead of RGBA in vaapi decoder (Closed)
Patch Set: decide scanout in runtime Created 3 years, 6 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/gfx/linux/client_native_pixmap_factory_dmabuf.cc
diff --git a/ui/gfx/linux/client_native_pixmap_factory_dmabuf.cc b/ui/gfx/linux/client_native_pixmap_factory_dmabuf.cc
index a91e50645fe50e971e647879c5ce6afb75296879..d92fc2cd4ef90bc907d424e93891cd16e742cab7 100644
--- a/ui/gfx/linux/client_native_pixmap_factory_dmabuf.cc
+++ b/ui/gfx/linux/client_native_pixmap_factory_dmabuf.cc
@@ -59,6 +59,9 @@ class ClientNativePixmapFactoryDmabuf : public ClientNativePixmapFactory {
format == gfx::BufferFormat::RGBX_8888 ||
format == gfx::BufferFormat::BGRA_8888 ||
format == gfx::BufferFormat::BGRX_8888 ||
+ // TODO(dshwang): add YUV_420_BIPLANAR to SCANOUT when cros
+ // kernel supports NV12 overlay. crbug.com/683347
+ format == gfx::BufferFormat::YUV_420_BIPLANAR ||
format == gfx::BufferFormat::YVU_420;
case gfx::BufferUsage::SCANOUT:
return format == gfx::BufferFormat::BGRX_8888 ||
@@ -78,6 +81,7 @@ class ClientNativePixmapFactoryDmabuf : public ClientNativePixmapFactory {
format == gfx::BufferFormat::R_8 ||
format == gfx::BufferFormat::RG_88 ||
#endif
+ format == gfx::BufferFormat::YUV_420_BIPLANAR ||
format == gfx::BufferFormat::BGRA_8888;
#else
return false;

Powered by Google App Engine
This is Rietveld 408576698