Adding GpuMemoryManager to track GpuCommandBufferStub visibility and last_used_time and dictate mem…

content/common/gpu/gpu_channel.h

Index: content/common/gpu/gpu_channel.h
diff --git a/content/common/gpu/gpu_channel.h b/content/common/gpu/gpu_channel.h
index 7fc6baf2b86bff1e284b4e3212278e2aacdc2ad3..91e49354003c96aa0656394ede16e4b1655f6738 100644
--- a/content/common/gpu/gpu_channel.h
+++ b/content/common/gpu/gpu_channel.h
@@ -16,6 +16,7 @@
 #include "base/process.h"
 #include "build/build_config.h"
 #include "content/common/gpu/gpu_command_buffer_stub.h"
+#include "content/common/gpu/gpu_memory_manager.h"
 #include "content/common/message_router.h"
 #include "ipc/ipc_sync_channel.h"
 #include "ui/gfx/gl/gl_share_group.h"
@@ -36,6 +37,7 @@ class WaitableEvent;
 // process. On the renderer side there's a corresponding GpuChannelHost.
 class GpuChannel : public IPC::Channel::Listener,
                    public IPC::Message::Sender,
+                   public GpuMemoryManagerClient,

[jonathan.backer, 2012/01/31 18:13:58]

Necessary?

[mmocny, 2012/01/31 18:54:57]

Indeed, no.
On 2012/01/31 18:13:58, jonathan.backer wrote:

                    public base::RefCountedThreadSafe<GpuChannel> {
  public:
   // Takes ownership of the renderer process handle.
@@ -73,6 +75,12 @@ class GpuChannel : public IPC::Channel::Listener,
   // IPC::Message::Sender implementation:
   virtual bool Send(IPC::Message* msg) OVERRIDE;
 
+  // GpuMemoryManagerClient implementation:
+  virtual void AppendAllCommandBufferStubs(

[nduca, 2012/01/31 06:53:47]

Overkill. Just append them all. Then filter then as a second pass inside the
consumer.

[mmocny, 2012/01/31 18:54:57]

Done.

+      std::vector<GpuMemoryManageableCommandBufferStub*>& stubs_with_surface,
+      std::vector<GpuMemoryManageableCommandBufferStub*>& stubs_without_surface)
+          OVERRIDE;
+
   // Whether this channel is able to handle IPC messages.
   bool IsScheduled();