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 {
 
 class GPU_EXPORT ImageFactory {
  public:
   ImageFactory();
 
-  // Returns a valid GpuMemoryBuffer format given a valid internalformat as
-  // defined by CHROMIUM_gpu_memory_buffer_image.
-  static gfx::GpuMemoryBuffer::Format ImageFormatToGpuMemoryBufferFormat(
-      unsigned internalformat);
+  // Returns the number of GpuMemoryBuffers used when given a valid
+  // |internalformat| as defined by CHROMIUM_gpu_memory_buffer_image.
+  static int GpuMemoryBufferCountForImageFormat(unsigned internalformat);
+
+  // Fills a vector of valid GpuMemoryBuffer formats in |formats| when given a
+  // valid |internalformat| as defined by CHROMIUM_gpu_memory_buffer_image.
+  static void ImageFormatToGpuMemoryBufferFormats(

[reveman, 2015/03/04 05:56:00]

How about avoiding the vector and instead add an 'int buffer' argument to the
ImageFormatToGpuMemoryBufferFormat function?

+      unsigned internalformat,
+      std::vector<gfx::GpuMemoryBuffer::Format>* formats);
 
   // 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,
       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,
       const gfx::Size& size,
       gfx::GpuMemoryBuffer::Format format,
       unsigned internalformat,
       int client_id) = 0;
 
  protected:
   virtual ~ImageFactory();
 };
 
 }  // namespace gpu
 
 #endif  // GPU_COMMAND_BUFFER_SERVICE_IMAGE_FACTORY_H_