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

Unified Diff: content/common/child_process_host_impl.cc

Issue 1415723006: ui: Add RGBX_8888 buffer format. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix gles but no conversion needed case Created 5 years, 2 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: content/common/child_process_host_impl.cc
diff --git a/content/common/child_process_host_impl.cc b/content/common/child_process_host_impl.cc
index 1ae2cc06437143803d0b12fe35796c8a5f774d2b..c9cf76bac9836f4b33952e5dd0d612b1634905cd 100644
--- a/content/common/child_process_host_impl.cc
+++ b/content/common/child_process_host_impl.cc
@@ -294,9 +294,8 @@ void ChildProcessHostImpl::OnAllocateGpuMemoryBuffer(
// AllocateForChildProcess() will check if |width| and |height| are valid
// and handle failure in a controlled way when not. We just need to make
- // sure |format| and |usage| are supported here.
- if (GpuMemoryBufferImplSharedMemory::IsFormatSupported(format) &&
- GpuMemoryBufferImplSharedMemory::IsUsageSupported(usage)) {
+ // sure |usage| is supported here.
+ if (GpuMemoryBufferImplSharedMemory::IsUsageSupported(usage)) {
*handle = GpuMemoryBufferImplSharedMemory::AllocateForChildProcess(
id, gfx::Size(width, height), format, peer_process_.Handle());
}

Powered by Google App Engine
This is Rietveld 408576698