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

content/common/gpu/gpu_memory_manager.h

+// Copyright (c) 2012 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.
+
+#ifndef CONTENT_COMMON_GPU_GPU_MEMORY_MANAGER_H_
+#define CONTENT_COMMON_GPU_GPU_MEMORY_MANAGER_H_
+#pragma once
+
+#if defined(ENABLE_GPU)
+
+#include "content/common/gpu/gpu_memory_management.h"
+
+////////////////////////////////////////////////////////////////////////////////

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

not sure we do /// delimiters?

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

I've seen it used, even inside content/common source files.

On 2012/01/31 06:53:47, nduca wrote:

+
+class GpuMemoryManager {
+public:
+  GpuMemoryManager(GpuMemoryManagerClient* client);
+  ~GpuMemoryManager();
+
+  void SetMemoryManagerClient(GpuMemoryManagerClient* client);
+
+  void ScheduleManage();
+  void Manage();

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

You might put a comment on Manage() like "// Public for test"

Or, just make it private and use friending.

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

Done.

+
+private:
+  GpuMemoryManagerClient* client_;
+  bool manage_scheduled_;
+
+  DISALLOW_COPY_AND_ASSIGN(GpuMemoryManager);
+};
+
+////////////////////////////////////////////////////////////////////////////////
+
+#endif
+
+#endif // CONTENT_COMMON_GPU_GPU_MEMORY_MANAGER_H_