gpu: Support other buffer targets for ScopedBufferBinder

gpu/command_buffer/service/context_state.cc

Index: gpu/command_buffer/service/context_state.cc
diff --git a/gpu/command_buffer/service/context_state.cc b/gpu/command_buffer/service/context_state.cc
index 7259d753b628349c4598f445b835408592d50610..28b48147ef79d6665fdb39fa45cc3fa1c3a2d3b6 100644
--- a/gpu/command_buffer/service/context_state.cc
+++ b/gpu/command_buffer/service/context_state.cc
@@ -77,16 +77,6 @@ bool TargetIsSupported(const FeatureInfo* feature_info, GLuint target) {
   }
 }
 
-bool BufferTargetIsSupported(GLuint target) {
-  switch (target) {
-    case GL_ARRAY_BUFFER:
-      return true;
-    default:
-      NOTREACHED();
-      return false;
-  }
-}
-
 GLuint GetBufferId(const Buffer* buffer) {
   if (buffer)
     return buffer->service_id();
@@ -317,8 +307,7 @@ void ContextState::DoLineWidth(GLfloat width) const {
 }
 
 void ContextState::RestoreBufferBinding(unsigned int target) const {

[piman, 2017/04/04 00:27:25]

If we are not going to implement the other targets, I would suggest renaming
this function to RestoreArrayBufferBinding, and removing the target parameter.

[Chandan, 2017/04/04 12:14:00]

I did not add other targets in BufferTargetIsSupported() to comply with
!state_restorer_ case in ScopedBufferBinder where only one target
GL_ARRAY_BUFFER is supported. 
We can definitely implement the other targets. I have uploaded patchset 2 with
support for other targets.

-  if (BufferTargetIsSupported(target))
-    glBindBuffer(target, GetBufferId(bound_array_buffer.get()));
+  glBindBuffer(target, GetBufferId(bound_array_buffer.get()));
 }
 
 void ContextState::RestoreBufferBindings() const {