Move the GL shader compilation step into GrGLShaderBuilder

Move the GL shader compilation step into GrGLShaderBuilder

Moves the compilation step and a few other blocks of code from
GrGLProgram to GrGLShaderBuilder. This way GrGLProgram doesn't have to
know whether or not there is a vertex shader.

Committed: http://code.google.com/p/skia/source/detail?r=11523

[bsalomon, 2013-09-18 20:03:25]

It seems like a step in the right direction. Eventually I'd like the builder to
do all code generation.

https://codereview.chromium.org/23533066/diff/1/src/gpu/gl/GrGLShaderBuilder.h
File src/gpu/gl/GrGLShaderBuilder.h (right):

https://codereview.chromium.org/23533066/diff/1/src/gpu/gl/GrGLShaderBuilder....
src/gpu/gl/GrGLShaderBuilder.h:238: * TODO: Hide these from the GrEffects using
friend or splitting this into two related classes.
Can we change this comment to say:

TODO: These are used by GrGLProgram to insert a mode color filter. Remove these
when the color filter is expressed as a GrEffect.

[Chris Dalton, 2013-09-18 20:17:07]

BTW, this depends on https://codereview.chromium.org/23464082/

[commit-bot: I haz the power, 2013-09-30 15:15:19]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[bsalomon, 2013-09-30 15:15:23]

On 2013/09/18 20:17:07, Chris Dalton wrote:
> BTW, this depends on https://codereview.chromium.org/23464082/

lgtm

[commit-bot: I haz the power, 2013-09-30 15:15:41]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/cdalton@nvidia.com/23533066/6002

[commit-bot: I haz the power, 2013-09-30 15:30:41]

Change committed as 11523