Cleanup GpuMemoryBuffer

content/common/gpu/client/gpu_memory_buffer_impl_surface_texture.cc

 // 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.
 
 #include "content/common/gpu/client/gpu_memory_buffer_impl_surface_texture.h"
 
 #include "base/atomic_sequence_num.h"
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "content/common/android/surface_texture_manager.h"
 #include "content/common/gpu/client/gpu_memory_buffer_factory_host.h"
 #include "ui/gl/gl_bindings.h"
 
 namespace content {
 namespace {
 
 base::StaticAtomicSequenceNumber g_next_buffer_id;
 
-void Noop() {
+void GpuMemoryBufferDeleted(const gfx::GpuMemoryBufferHandle& handle,
+                            uint32 sync_point) {
+  GpuMemoryBufferFactoryHost::GetInstance()->DestroyGpuMemoryBuffer(handle,
+                                                                    sync_point);
 }
 
 void GpuMemoryBufferCreated(
     const gfx::Size& size,
     gfx::GpuMemoryBuffer::Format format,
     const GpuMemoryBufferImpl::CreationCallback& callback,
     const gfx::GpuMemoryBufferHandle& handle) {
   DCHECK_EQ(gfx::SURFACE_TEXTURE_BUFFER, handle.type);
 
   callback.Run(GpuMemoryBufferImplSurfaceTexture::CreateFromHandle(
-      handle, size, format, base::Bind(&Noop)));
+      handle, size, format, base::Bind(&GpuMemoryBufferDeleted, handle)));
 }
 
 void GpuMemoryBufferCreatedForChildProcess(
     const GpuMemoryBufferImpl::AllocationCallback& callback,
     const gfx::GpuMemoryBufferHandle& handle) {
   DCHECK_EQ(gfx::SURFACE_TEXTURE_BUFFER, handle.type);
 
   callback.Run(handle);
 }
 
@@ skipping 68 matching lines @@
 
   ANativeWindow_setBuffersGeometry(
       native_window, size.width(), size.height(), WindowFormat(format));
 
   return make_scoped_ptr<GpuMemoryBufferImpl>(
       new GpuMemoryBufferImplSurfaceTexture(
           size, format, callback, handle.global_id, native_window));
 }
 
 // static
+void GpuMemoryBufferImplSurfaceTexture::DeletedByChildProcess(
+    const gfx::GpuMemoryBufferId& id,
+    uint32_t sync_point) {
+  gfx::GpuMemoryBufferHandle handle;
+  handle.type = gfx::SURFACE_TEXTURE_BUFFER;
+  handle.global_id = id;
+

[reveman, 2014/11/04 00:16:13]

nit: make blank line usage consistent with Create/AllocateForChildProcess

+  GpuMemoryBufferFactoryHost::GetInstance()->DestroyGpuMemoryBuffer(handle,
+                                                                    sync_point);
+}
+
+// static
 bool GpuMemoryBufferImplSurfaceTexture::IsFormatSupported(Format format) {
   switch (format) {
     case RGBA_8888:
       return true;
     case RGBX_8888:
     case BGRA_8888:
       return false;
   }
 
   NOTREACHED();
@@ skipping 66 matching lines @@
 
 gfx::GpuMemoryBufferHandle GpuMemoryBufferImplSurfaceTexture::GetHandle()
     const {
   gfx::GpuMemoryBufferHandle handle;
   handle.type = gfx::SURFACE_TEXTURE_BUFFER;
   handle.global_id = id_;
   return handle;
 }
 
 }  // namespace content