Cleanup GpuMemoryBuffer

content/common/gpu/client/gpu_channel_host.h

Index: content/common/gpu/client/gpu_channel_host.h
diff --git a/content/common/gpu/client/gpu_channel_host.h b/content/common/gpu/client/gpu_channel_host.h
index d20d5222bfa52535ad051b5bd3848091bbdcbb0e..4bfe7f8d8ea099aefe513dd35bb78265a6602bef 100644
--- a/content/common/gpu/client/gpu_channel_host.h
+++ b/content/common/gpu/client/gpu_channel_host.h
@@ -161,6 +161,10 @@ class GpuChannelHost : public IPC::Sender,
   // Generate a route ID guaranteed to be unique for this channel.
   int32 GenerateRouteID();
 
+  // To be called on IO thread.

[reveman, 2014/10/21 20:27:58]

I assume it's more appropriate to do the thread hop in this implementation than
requiring it to be done by the caller, correct?

[alexst (slow to review), 2014/10/22 14:47:08]

I was initially thinking that since the browser side already does this on IO, it
was reasonably to enforce, but it occurs to me that we don't know which thread
might call it in a renderer, so the restriction is probably invalid.

+  void WaitForPendingGpuMemoryBufferUsageToComplete(
+      const base::Closure& callback);
+
  private:
   friend class base::RefCountedThreadSafe<GpuChannelHost>;
   GpuChannelHost(GpuChannelHostFactory* factory,
@@ -242,6 +246,16 @@ class GpuChannelHost : public IPC::Sender,
   typedef base::hash_map<int, CommandBufferProxyImpl*> ProxyMap;
   ProxyMap proxies_;
 
+  void WaitForPendingGpuMemoryBufferUsageToCompleteOnMain(uint32 id);
+  void OnSignalSyncPoint(uint32 id);
+  void ProcessSignalCounterOnIO(uint32 id);
+
+  typedef base::hash_map<uint32, base::Closure> SignalTaskMap;
+  typedef base::hash_map<uint32, int> SignalCounterMap;
+  uint32 next_signal_id_;
+  SignalTaskMap signal_tasks_;
+  SignalCounterMap signal_counters_;
+
   DISALLOW_COPY_AND_ASSIGN(GpuChannelHost);
 };