Make program unit test run clean and add it to tests program

[bsalomon, 2011-08-15 20:42:52]

Fixes up some issues in the shader generator to make the shader unit test run
cleanly.

I've been meaning to make the unit test able to run tests that require a GL
context for a long time. This change creates a global GL context and Gr context
available for unit tests. It also adds a test that runs the GrGpuGLShaders
program unit test.

[epoger, 2011-08-15 21:31:42]

LGTM with some optional suggestions

http://codereview.appspot.com/4898049/diff/1/gpu/src/GrGLProgram.cpp
File gpu/src/GrGLProgram.cpp (right):

http://codereview.appspot.com/4898049/diff/1/gpu/src/GrGLProgram.cpp#newcode1273
gpu/src/GrGLProgram.cpp:1273: segments->fFSCode.appendf("\tvec4 %s = vec4(0, 0,
0, 0);\n", sumVar.c_str());
please wrap at 80 characters if it doesn't get in the way too much

http://codereview.appspot.com/4898049/diff/1/gyp/tests.gyp
File gyp/tests.gyp (right):

http://codereview.appspot.com/4898049/diff/1/gyp/tests.gyp#newcode13
gyp/tests.gyp:13: '../gpu/include',
I don't think you need to add these... depending on gpu.gyp should provide them
automatically.  Please give it a try without them and see if it works...

[bsalomon, 2011-08-15 21:43:21]

http://codereview.appspot.com/4898049/diff/1/gpu/src/GrGLProgram.cpp
File gpu/src/GrGLProgram.cpp (right):

http://codereview.appspot.com/4898049/diff/1/gpu/src/GrGLProgram.cpp#newcode1273
gpu/src/GrGLProgram.cpp:1273: segments->fFSCode.appendf("\tvec4 %s = vec4(0, 0,
0, 0);\n", sumVar.c_str());
On 2011/08/15 21:31:42, epoger wrote:
> please wrap at 80 characters if it doesn't get in the way too much

Done.

http://codereview.appspot.com/4898049/diff/1/gyp/tests.gyp
File gyp/tests.gyp (right):

http://codereview.appspot.com/4898049/diff/1/gyp/tests.gyp#newcode13
gyp/tests.gyp:13: '../gpu/include',
On 2011/08/15 21:31:42, epoger wrote:
> I don't think you need to add these... depending on gpu.gyp should provide
them
> automatically.  Please give it a try without them and see if it works...

I don't need the gpu/include path but I do need gpu/src. The tests need headers
in src that aren't normally public.

[Stephen White, 2011-08-15 21:46:49]

Cool!  LGTM.

[bsalomon, 2011-08-16 15:46:27]

Closed with r2121