GPU client side changes for GpuMemoryBuffers

gpu/command_buffer/client/gpu_memory_buffer_tracker.h

+// Copyright (c) 2013 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 GPU_COMMAND_BUFFER_CLIENT_GPU_MEMORY_BUFFER_TRACKER_H_
+#define GPU_COMMAND_BUFFER_CLIENT_GPU_MEMORY_BUFFER_TRACKER_H_
+
+#include <GLES2/gl2.h>
+
+#include "../client/hash_tables.h"
+#include "../common/gles2_cmd_format.h"
+#include "base/basictypes.h"
+#include "gles2_impl_export.h"
+
+namespace gpu {
+class GpuMemoryBuffer;
+
+namespace gles2 {
+class GpuMemoryBufferFactory;
+
+// Tracks GPU memory buffer objects for client side of command buffer.
+class GLES2_IMPL_EXPORT GpuMemoryBufferTracker {
+ public:
+  explicit GpuMemoryBufferTracker(GpuMemoryBufferFactory* factory);

[joth, 2013/05/03 02:04:11]

This is taking ownership of |factory| ? if so pass it in and store it as
scoped_ptr<>

[kaanb, 2013/05/03 02:37:31]

GLES2Implementation is taking ownership of it. I made it a scoped_ptr<> there.

+  ~GpuMemoryBufferTracker();
+
+  GpuMemoryBuffer* CreateBuffer(GLuint image_id, GLsizei width, GLsizei height);
+  GpuMemoryBuffer* GetBuffer(GLuint image_id);
+  void RemoveBuffer(GLuint image_id);
+
+ private:
+  typedef gpu::hash_map<GLuint, GpuMemoryBuffer*> BufferMap;
+  BufferMap buffers_;
+  GpuMemoryBufferFactory* factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(GpuMemoryBufferTracker);
+};
+
+}  // namespace gles2
+}  // namespace gpu
+
+#endif  // GPU_COMMAND_BUFFER_CLIENT_GPU_MEMORY_BUFFER_TRACKER_H_