Improve BindBufferBase/BindBufferRange and a few other commands.

gpu/command_buffer/client/share_group.cc

Index: gpu/command_buffer/client/share_group.cc
diff --git a/gpu/command_buffer/client/share_group.cc b/gpu/command_buffer/client/share_group.cc
index 726714f2e2e4bf9d52f5b156ba20cc78d829a70d..755614b3a5099560854b110086f6b9865a8fd71d 100644
--- a/gpu/command_buffer/client/share_group.cc
+++ b/gpu/command_buffer/client/share_group.cc
@@ -76,6 +76,28 @@ class IdHandler : public IdHandlerInterface {
     (gl_impl->*bind_fn)(target, id);
     return result;
   }
+  bool MarkAsUsedForBind(GLES2Implementation* gl_impl,
+                         GLenum target,
+                         GLuint index,
+                         GLuint id,
+                         BindIndexedFn bind_fn) override {
+    base::AutoLock auto_lock(lock_);
+    bool result = id ? id_allocator_.MarkAsUsed(id) : true;
+    (gl_impl->*bind_fn)(target, index, id);
+    return result;
+  }
+  bool MarkAsUsedForBind(GLES2Implementation* gl_impl,
+                         GLenum target,
+                         GLuint index,
+                         GLuint id,
+                         GLintptr offset,
+                         GLsizeiptr size,
+                         BindIndexedRangeFn bind_fn) override {
+    base::AutoLock auto_lock(lock_);
+    bool result = id ? id_allocator_.MarkAsUsed(id) : true;
+    (gl_impl->*bind_fn)(target, index, id, offset, size);
+    return result;
+  }
 
   void FreeContext(GLES2Implementation* gl_impl) override {}
 
@@ -166,6 +188,42 @@ class StrictIdHandler : public IdHandlerInterface {
     (gl_impl->*bind_fn)(target, id);
     return true;
   }
+  bool MarkAsUsedForBind(GLES2Implementation* gl_impl,
+                         GLenum target,
+                         GLuint index,
+                         GLuint id,
+                         BindIndexedFn bind_fn) override {
+#ifndef NDEBUG
+    if (id != 0) {
+      base::AutoLock auto_lock(lock_);
+      DCHECK(id_states_[id - 1] == kIdInUse);
+    }
+#endif
+    // StrictIdHandler is used if |bind_generates_resource| is false. In that
+    // case, |bind_fn| will not use Flush() after helper->Bind*(), so it is OK
+    // to call |bind_fn| without holding the lock.
+    (gl_impl->*bind_fn)(target, index, id);
+    return true;
+  }
+  bool MarkAsUsedForBind(GLES2Implementation* gl_impl,
+                         GLenum target,
+                         GLuint index,
+                         GLuint id,
+                         GLintptr offset,
+                         GLsizeiptr size,
+                         BindIndexedRangeFn bind_fn) override {
+#ifndef NDEBUG
+    if (id != 0) {
+      base::AutoLock auto_lock(lock_);
+      DCHECK(id_states_[id - 1] == kIdInUse);
+    }
+#endif
+    // StrictIdHandler is used if |bind_generates_resource| is false. In that
+    // case, |bind_fn| will not use Flush() after helper->Bind*(), so it is OK
+    // to call |bind_fn| without holding the lock.
+    (gl_impl->*bind_fn)(target, index, id, offset, size);
+    return true;
+  }
 
   // Overridden from IdHandlerInterface.
   void FreeContext(GLES2Implementation* gl_impl) override {
@@ -235,6 +293,24 @@ class NonReusedIdHandler : public IdHandlerInterface {
     // This is only used for Shaders and Programs which have no bind.
     return false;
   }
+  bool MarkAsUsedForBind(GLES2Implementation* /* gl_impl */,
+                         GLenum /* target */,
+                         GLuint /* index */,
+                         GLuint /* id */,
+                         BindIndexedFn /* bind_fn */) override {
+    // This is only used for Shaders and Programs which have no bind.
+    return false;
+  }
+  bool MarkAsUsedForBind(GLES2Implementation* /* gl_impl */,
+                         GLenum /* target */,
+                         GLuint /* index */,
+                         GLuint /* id */,
+                         GLintptr /* offset */,
+                         GLsizeiptr /* size */,
+                         BindIndexedRangeFn /* bind_fn */) override {
+    // This is only used for Shaders and Programs which have no bind.
+    return false;
+  }
 
   void FreeContext(GLES2Implementation* gl_impl) override {}