Change CHROMIUM_image declarations to support multi planar input.

gpu/command_buffer/service/image_factory.h

 // Copyright 2014 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.
 
 #ifndef GPU_COMMAND_BUFFER_SERVICE_IMAGE_FACTORY_H_
 #define GPU_COMMAND_BUFFER_SERVICE_IMAGE_FACTORY_H_
 
 #include "base/memory/ref_counted.h"
 #include "gpu/gpu_export.h"
 #include "ui/gfx/geometry/size.h"
 #include "ui/gfx/gpu_memory_buffer.h"
 
 namespace gfx {
 class GLImage;
 }
 
 namespace gpu {
 struct Capabilities;
 
 class GPU_EXPORT ImageFactory {
  public:
   ImageFactory();
 
-  // Returns a valid GpuMemoryBuffer format given a valid internalformat as
-  // defined by CHROMIUM_gpu_memory_buffer_image.
+  // Returns the number of GpuMemoryBuffers used when given a valid
+  // |internalformat| as defined by CHROMIUM_gpu_memory_buffer_image.
+  static int GpuMemoryBufferCountForImageFormat(unsigned internalformat);

[reveman, 2015/03/12 19:37:23]

size_t as return type? And how about renaming this to
NumberOfPlanesForImageFormat?

[emircan, 2015/03/12 22:34:26]

Done.

+
+  // Returns a valid GpuMemoryBuffer format given a valid |internalformat| as
+  // defined by CHROMIUM_gpu_memory_buffer_image, and |buffer_index|
+  // representing the zero-based index of the buffer in a multiple planar
+  // format.
   static gfx::GpuMemoryBuffer::Format ImageFormatToGpuMemoryBufferFormat(
-      unsigned internalformat);
+      unsigned internalformat,
+      int buffer_index);

[reveman, 2015/03/12 19:37:23]

"size_t plane"?

[emircan, 2015/03/12 22:34:26]

Done.

 
   // Returns a valid GpuMemoryBuffer usage given a valid usage as defined by
   // CHROMIUM_gpu_memory_buffer_image.
   static gfx::GpuMemoryBuffer::Usage ImageUsageToGpuMemoryBufferUsage(
       unsigned usage);
 
   // Returns true if |internalformat| is compatible with |format|.
   static bool IsImageFormatCompatibleWithGpuMemoryBufferFormat(
       unsigned internalformat,
+      int buffer_index,

[reveman, 2015/03/12 19:37:23]

size_t plane

[emircan, 2015/03/12 22:34:26]

Done.

       gfx::GpuMemoryBuffer::Format format);
 
   // Returns true if |format| is supported by |capabilities|.
   static bool IsGpuMemoryBufferFormatSupported(
       gfx::GpuMemoryBuffer::Format format,
       const Capabilities& capabilities);
 
   // Returns true if |size| is valid for |format|.
   static bool IsImageSizeValidForGpuMemoryBufferFormat(
       const gfx::Size& size,
       gfx::GpuMemoryBuffer::Format format);
 
   // Creates a GLImage instance for GPU memory buffer identified by |handle|.
   // |client_id| should be set to the client requesting the creation of instance
   // and can be used by factory implementation to verify access rights.
-  virtual scoped_refptr<gfx::GLImage> CreateImageForGpuMemoryBuffer(
-      const gfx::GpuMemoryBufferHandle& handle,
+  virtual scoped_refptr<gfx::GLImage> CreateImageForGpuMemoryBuffers(
+      const std::vector<gfx::GpuMemoryBufferHandle>& handles,
       const gfx::Size& size,
-      gfx::GpuMemoryBuffer::Format format,
+      const std::vector<gfx::GpuMemoryBuffer::Format>& formats,
       unsigned internalformat,
       int client_id) = 0;
 
  protected:
   virtual ~ImageFactory();
 };
 
 }  // namespace gpu
 
 #endif  // GPU_COMMAND_BUFFER_SERVICE_IMAGE_FACTORY_H_