ozone: Implement zero/one-copy texture for Ozone GBM.

content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_buffer.h

Index: content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_buffer.h
diff --git a/content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_buffer.h b/content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_buffer.h
index 040eab9738dc7160d706f3f3dbb0464316c7969f..23ad741778d151359b5fdc6bc8fe653a7cd41c80 100644
--- a/content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_buffer.h
+++ b/content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_buffer.h
@@ -7,6 +7,8 @@
 
 #include "content/common/gpu/client/gpu_memory_buffer_impl.h"
 
+struct gbm_bo;
+
 namespace content {
 
 // Implementation of GPU memory buffer based on Ozone native buffers.
@@ -18,6 +20,8 @@ class GpuMemoryBufferImplOzoneNativeBuffer : public GpuMemoryBufferImpl {
       Format format,
       const DestructionCallback& callback);
 
+  ~GpuMemoryBufferImplOzoneNativeBuffer() override;
+
   // Overridden from gfx::GpuMemoryBuffer:
   bool Map(void** data) override;
   void Unmap() override;
@@ -29,7 +33,14 @@ class GpuMemoryBufferImplOzoneNativeBuffer : public GpuMemoryBufferImpl {
                                        const gfx::Size& size,
                                        Format format,
                                        const DestructionCallback& callback);
-  ~GpuMemoryBufferImplOzoneNativeBuffer() override;
+
+  bool Initialize(const gfx::GpuMemoryBufferHandle& handle);
+
+  uint32_t vgem_bo_handle_;
+  base::ScopedFD dma_buf_;
+  base::ScopedFD vgem_fd_;
+  size_t stride_;
+  void* mmap_ptr_;

[reveman, 2015/05/14 13:22:03]

Please move all this into ui/ozone/ and keep one opaque object here instead.
GpuMemoryBufferImplOzoneNativeBuffer is supposed to be a thin wrapper around an
ozone implementation and not contain any real logic.

Maybe use a NativePixmap here on the client side too or create some new type of
object.

SurfaceFactoryOzone::CreateNativePixmapFromHandle(base::FileDescriptor fd)
maybe..

[dshwang, 2015/05/14 14:58:15]

now I understand what you mean. It's very reasonable. I'll do it soon.

 
   DISALLOW_COPY_AND_ASSIGN(GpuMemoryBufferImplOzoneNativeBuffer);
 };