content: Cleanup GpuMemoryBufferImpl destruction.

content/common/gpu/client/gpu_memory_buffer_impl.h

Index: content/common/gpu/client/gpu_memory_buffer_impl.h
diff --git a/content/common/gpu/client/gpu_memory_buffer_impl.h b/content/common/gpu/client/gpu_memory_buffer_impl.h
index 2da6b437e8ac60d44a70e441632fafb29918cfc8..0beab681883516a337d832f3413fc8ee51acffb0 100644
--- a/content/common/gpu/client/gpu_memory_buffer_impl.h
+++ b/content/common/gpu/client/gpu_memory_buffer_impl.h
@@ -19,6 +19,7 @@ class GpuMemoryBufferImpl : public gfx::GpuMemoryBuffer {
       CreationCallback;
   typedef base::Callback<void(const gfx::GpuMemoryBufferHandle& handle)>
       AllocationCallback;
+  typedef base::Closure DestructionCallback;
 
   virtual ~GpuMemoryBufferImpl();
 
@@ -50,7 +51,8 @@ class GpuMemoryBufferImpl : public gfx::GpuMemoryBuffer {
   static scoped_ptr<GpuMemoryBufferImpl> CreateFromHandle(
       const gfx::GpuMemoryBufferHandle& handle,
       const gfx::Size& size,
-      unsigned internalformat);
+      unsigned internalformat,
+      const DestructionCallback& callback);

[piman, 2014/09/24 03:19:10]

nit: I think it should be documented explicitly that the callback is called on
the thread that destroys the GpuMemoryBufferImpl, not the thread that called
CreateFromHandle

[reveman, 2014/09/24 06:03:38]

Done.

 
   // Returns true if |internalformat| is a format recognized by this base class.
   static bool IsFormatValid(unsigned internalformat);