Added support for glGetString, glGetShaderSource,...

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


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

[apatrick_chromium, 2010-02-27 00:21:34]

lgtm

http://codereview.chromium.org/661220/diff/1/19
File gpu/command_buffer/build_gles2_cmd_buffer.py (right):

http://codereview.chromium.org/661220/diff/1/19#newcode1396
gpu/command_buffer/build_gles2_cmd_buffer.py:1396: test =
self._remove_expected_call_re.sub('', test)
How about not generating the expected calls in the first place?

http://codereview.chromium.org/661220/diff/1/16
File gpu/command_buffer/client/gles2_implementation.cc (right):

http://codereview.chromium.org/661220/diff/1/16#newcode375
gpu/command_buffer/client/gles2_implementation.cc:375: if (level < 0 || height <
0 || width < 0) {
Random thought... If we're at all concerned about performance, we could do this:
((level | height | width) < 0). Just thought I'd mention it while I remember.

http://codereview.chromium.org/661220/diff/1/11
File gpu/command_buffer/common/gles2_cmd_utils.cc (right):

http://codereview.chromium.org/661220/diff/1/11#newcode404
gpu/command_buffer/common/gles2_cmd_utils.cc:404: uint32
GLES2Util::GLErrorToErrorBit(uint32 error) {
These GL error constants are all in the range 0x500 to 0x506. You could just
do:
uint32 bit = error - GL_INVALID_ENUM;
DCHECK(bit < 32);
return bit;

http://codereview.chromium.org/661220/diff/1/12
File gpu/command_buffer/common/gles2_cmd_utils.h (right):

http://codereview.chromium.org/661220/diff/1/12#newcode17
gpu/command_buffer/common/gles2_cmd_utils.h:17: namespace GLErrorBit {
lower case / underscores for namespaces?

http://codereview.chromium.org/661220/diff/1/8
File gpu/command_buffer/service/gles2_cmd_decoder_unittest_1_autogen.h (right):

http://codereview.chromium.org/661220/diff/1/8#newcode1122
gpu/command_buffer/service/gles2_cmd_decoder_unittest_1_autogen.h:1122:
kServiceProgramId, GL_INFO_LOG_LENGTH, _))    .WillOnce(
This is in a funny place.