gpu: Add IOSurface backed GpuMemoryBuffer implementation.

content/common/gpu/client/gpu_memory_buffer_impl.h

 // Copyright 2013 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 CONTENT_COMMON_GPU_CLIENT_GPU_MEMORY_BUFFER_IMPL_H_
 #define CONTENT_COMMON_GPU_CLIENT_GPU_MEMORY_BUFFER_IMPL_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "ui/gfx/gpu_memory_buffer.h"
 #include "ui/gfx/size.h"
 
 namespace content {
 
-// Provides common implementation of a GPU memory buffer based
-// on a shared memory handle.
+// Provides common implementation of a GPU memory buffer.
 class GpuMemoryBufferImpl : public gfx::GpuMemoryBuffer {
  public:
-  GpuMemoryBufferImpl(scoped_ptr<base::SharedMemory> shared_memory,
-                      size_t width,
-                      size_t height,
-                      unsigned internalformat);
+  static scoped_ptr<GpuMemoryBufferImpl> Create(
+      gfx::GpuMemoryBufferHandle handle,
+      gfx::Size size,
+      unsigned internalformat);
+
   virtual ~GpuMemoryBufferImpl();
 
-  // Overridden from gfx::GpuMemoryBuffer:
-  virtual void Map(AccessMode mode, void** vaddr) OVERRIDE;
-  virtual void Unmap() OVERRIDE;
-  virtual bool IsMapped() const OVERRIDE;
-  virtual uint32 GetStride() const OVERRIDE;
-  virtual gfx::GpuMemoryBufferHandle GetHandle() const OVERRIDE;
-
   static bool IsFormatValid(unsigned internalformat);

[piman, 2013/12/03 02:59:51]

This gets hidden by the GpuMemoryBufferImplIOSurface version. Should it move
(with the DCHECK) to GpuMemoryBufferImplShm?

[reveman, 2013/12/03 17:58:32]

I think this belongs in GpuMemoryBufferImpl as it's used by
RenderThreadImpl::AllocateGpuMemoryBuffer to verify that format is a generally
valid format.

I renamed the GpuMemoryBufferImplIOSurface version to IsFormatSupported for this
to be less confusing.

   static size_t BytesPerPixel(unsigned internalformat);
 
- private:
-  scoped_ptr<base::SharedMemory> shared_memory_;
+  // Overridden from gfx::GpuMemoryBuffer:
+  virtual bool IsMapped() const OVERRIDE;
+  virtual uint32 GetStride() const OVERRIDE;
+
+ protected:
+  GpuMemoryBufferImpl(gfx::Size size, unsigned internalformat);
+
   const gfx::Size size_;
   unsigned internalformat_;
   bool mapped_;
-
-  DISALLOW_COPY_AND_ASSIGN(GpuMemoryBufferImpl);

[piman, 2013/12/03 02:59:51]

nit: keep

[reveman, 2013/12/03 17:58:32]

Done.

 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_GPU_CLIENT_GPU_MEMORY_BUFFER_IMPL_H_