"/b/slave/chromium-rel-linux/build/src/gpu/command_buffer/client/gles2_demo_c...

"/b/slave/chromium-rel-linux/build/src/gpu/command_buffer/client/gles2_demo_cc.cc:50:warning: 'length' may be used uninitialized in this function"

Revert 40193 - Added support for glGetString, glGetShaderSource,
glGetProgramInfoLog and glGetShaderInfoLog

This turned up lots of issues.

* glGetShaderiv needed to be wrapped so it 
  returns the size of the shader the user passed
  in. Not the munged shader sent to GL.

* Another issue that came up is that client
  side validation is needed to ensure that
  the client side GLES2Implementation does not
  generate bad commands.  For example, calling
  glUniform1iv with a negative count would
  generate a bad command. The service side
  would see that and think the program was
  malicious and stop processing commands.

  So, that means I had to add some client side
  validation and wrap glGetError on the client 
  side so I can synthisize GL errors there.

TEST=lots of unit tests.
BUG=none

Review URL: http://codereview.chromium.org/661220

TBR=gman@chromium.org

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=40198