Change CHROMIUM_image declarations to support multi planar input.

content/common/gpu/client/command_buffer_proxy_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/common/gpu/client/command_buffer_proxy_impl.h"
 
 #include "base/callback.h"
 #include "base/logging.h"
 #include "base/memory/shared_memory.h"
 #include "base/stl_util.h"
@@ skipping 329 matching lines @@
   if (last_state_.error != gpu::error::kNoError)
     return;
 
   Send(new GpuCommandBufferMsg_DestroyTransferBuffer(route_id_, id));
 }
 
 gpu::Capabilities CommandBufferProxyImpl::GetCapabilities() {
   return capabilities_;
 }
 
-int32_t CommandBufferProxyImpl::CreateImage(ClientBuffer buffer,
+int32_t CommandBufferProxyImpl::CreateImage(ClientBuffer* const buffers,
                                             size_t width,
                                             size_t height,
                                             unsigned internalformat) {
   CheckLock();
   if (last_state_.error != gpu::error::kNoError)
     return -1;
 
   int32 new_id = channel_->ReserveImageId();
 
   gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager =
       channel_->gpu_memory_buffer_manager();
-  gfx::GpuMemoryBuffer* gpu_memory_buffer =
-      gpu_memory_buffer_manager->GpuMemoryBufferFromClientBuffer(buffer);
-  DCHECK(gpu_memory_buffer);
 
-  // This handle is owned by the GPU process and must be passed to it or it
-  // will leak. In otherwords, do not early out on error between here and the
-  // sending of the CreateImage IPC below.
+  // Log and return if the |internalformat| isn't supported.
+  if (!gpu::ImageFactory::IsImageFormatSupported(internalformat)) {

[reveman, 2015/03/18 16:59:00]

nit: this is on the child process side so you can just
DCHECK(gpu::ImageFactory::IsImageFormatSupported(internalformat)) here instead
as that's more useful for debugging purposes when you want to know asap when
something is wrong

[emircan, 2015/03/18 23:51:26]

Done.

+    LOG(ERROR) << "Internalformat is not supported.";
+    return -1;
+  }
+
+  // Check the buffer count for the given |internalformat| and initialize the
+  // vectors where data will be passed. Log and return if the |internalformat|
+  // isn't supported.
+  size_t num_buffers =
+      gpu::ImageFactory::NumberOfPlanesForImageFormat(internalformat);
+  std::vector<gfx::GpuMemoryBufferHandle> handles;
+  std::vector<gfx::GpuMemoryBuffer::Format> formats;
   bool requires_sync_point = false;
-  gfx::GpuMemoryBufferHandle handle =
-      channel_->ShareGpuMemoryBufferToGpuProcess(gpu_memory_buffer->GetHandle(),
-                                                 &requires_sync_point);
 
-  DCHECK(gpu::ImageFactory::IsImageSizeValidForGpuMemoryBufferFormat(
-      gfx::Size(width, height), gpu_memory_buffer->GetFormat()));
-  DCHECK(gpu::ImageFactory::IsImageFormatCompatibleWithGpuMemoryBufferFormat(
-      internalformat, gpu_memory_buffer->GetFormat()));
+  for (size_t i = 0; i < num_buffers; ++i) {
+    gfx::GpuMemoryBuffer* gpu_memory_buffer =
+        gpu_memory_buffer_manager->GpuMemoryBufferFromClientBuffer(buffers[i]);
+    DCHECK(gpu_memory_buffer);
+
+    formats.push_back(gpu_memory_buffer->GetFormat());
+    DCHECK(gpu::ImageFactory::IsImageSizeValidForGpuMemoryBufferFormat(
+        gfx::Size(width, height), formats[i]));
+    DCHECK(gpu::ImageFactory::IsImageFormatCompatibleWithGpuMemoryBufferFormat(
+        internalformat, i, formats[i]));
+
+    bool buffer_requires_sync_point = false;
+    // This handle is owned by the GPU process and must be passed to it or it
+    // will leak. In other words, do not early out on error between here and the
+    // sending of the CreateImage IPC below.
+    handles.push_back(channel_->ShareGpuMemoryBufferToGpuProcess(
+        gpu_memory_buffer->GetHandle(), &buffer_requires_sync_point));
+
+    // We set a destruction sync point on all buffers if one happen to require
+    // one.
+    requires_sync_point |= buffer_requires_sync_point;
+  }
+
   if (!Send(new GpuCommandBufferMsg_CreateImage(route_id_,
                                                 new_id,
-                                                handle,
+                                                handles,
                                                 gfx::Size(width, height),
-                                                gpu_memory_buffer->GetFormat(),
+                                                formats,
                                                 internalformat))) {
     return -1;
   }
 
+  uint32 sync_point = InsertSyncPoint();

[reveman, 2015/03/18 16:59:00]

Please move this inside the "if (requires_sync_point)" scope. It's expensive so
we should avoid it unless necessary.

[emircan, 2015/03/18 23:51:26]

Done. Sorry for missing that.

   if (requires_sync_point) {
-    gpu_memory_buffer_manager->SetDestructionSyncPoint(gpu_memory_buffer,
-                                                       InsertSyncPoint());
+    for (size_t i = 0; i < num_buffers; ++i) {
+      gfx::GpuMemoryBuffer* gpu_memory_buffer =
+          gpu_memory_buffer_manager->GpuMemoryBufferFromClientBuffer(
+              buffers[i]);
+      gpu_memory_buffer_manager->SetDestructionSyncPoint(gpu_memory_buffer,
+                                                         sync_point);
+    }
   }
 
   return new_id;
 }
 
 void CommandBufferProxyImpl::DestroyImage(int32 id) {
   CheckLock();
   if (last_state_.error != gpu::error::kNoError)
     return;
 
   Send(new GpuCommandBufferMsg_DestroyImage(route_id_, id));
 }
 
 int32_t CommandBufferProxyImpl::CreateGpuMemoryBufferImage(
     size_t width,
     size_t height,
     unsigned internalformat,
     unsigned usage) {
   CheckLock();
-  scoped_ptr<gfx::GpuMemoryBuffer> buffer(
-      channel_->gpu_memory_buffer_manager()->AllocateGpuMemoryBuffer(
-          gfx::Size(width, height),
-          gpu::ImageFactory::ImageFormatToGpuMemoryBufferFormat(internalformat),
-          gpu::ImageFactory::ImageUsageToGpuMemoryBufferUsage(usage)));
-  if (!buffer)
+
+  size_t num_buffers =
+      gpu::ImageFactory::NumberOfPlanesForImageFormat(internalformat);
+  if (num_buffers < 1) {

[reveman, 2015/03/18 16:59:00]

Please DCHECK(gpu::ImageFactory::IsImageFormatSupported(internalformat))
instead.

[emircan, 2015/03/18 23:51:26]

Done.

+    LOG(ERROR) << "Internalformat is not supported.";
     return -1;
+  }
 
-  return CreateImage(buffer->AsClientBuffer(), width, height, internalformat);
+  ScopedVector<gfx::GpuMemoryBuffer> buffers;
+  std::vector<ClientBuffer> client_buffers;
+  for (size_t i = 0; i < num_buffers; ++i) {
+    gfx::GpuMemoryBuffer::Format format =
+        gpu::ImageFactory::ImageFormatToGpuMemoryBufferFormat(internalformat,
+                                                              i);
+    buffers.push_back(
+        channel_->gpu_memory_buffer_manager()->AllocateGpuMemoryBuffer(
+            gfx::Size(width, height), format,
+            gpu::ImageFactory::ImageUsageToGpuMemoryBufferUsage(usage)));
+    if (!buffers[i])
+      return -1;
+
+    client_buffers[i] = buffers[i]->AsClientBuffer();
+  }
+  return CreateImage(client_buffers.data(), width, height, internalformat);
 }
 
 int CommandBufferProxyImpl::GetRouteID() const {
   return route_id_;
 }
 
 uint32 CommandBufferProxyImpl::CreateStreamTexture(uint32 texture_id) {
   CheckLock();
   if (last_state_.error != gpu::error::kNoError)
     return 0;
@@ skipping 176 matching lines @@
   }
 }
 
 void CommandBufferProxyImpl::OnUpdateVSyncParameters(base::TimeTicks timebase,
                                                      base::TimeDelta interval) {
   if (!update_vsync_parameters_completion_callback_.is_null())
     update_vsync_parameters_completion_callback_.Run(timebase, interval);
 }
 
 }  // namespace content