Move state onto GrGLShaderBuilder: sampleCoords + complexCoord

1- and 2-point RadialGradient mappings need to write these, so we move them onto the state.

Uglifies a lot of genStageCode, since what were two stack vars end up on an object. In the fairly short run, though, most (?) of those references ought to migrate into CustomStage objects.

[TomH, 2012-05-17 19:44:41]

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

http://codereview.appspot.com/6216057/diff/1/src/gpu/gl/GrGLProgram.cpp#newco...
src/gpu/gl/GrGLProgram.cpp:1300: GrGLShaderVar* radial2Params) {
Effectively, fsCoordName was being used as an input, and sampleCoords as an
output; now they both are in
segments->fSampleCoords.

http://codereview.appspot.com/6216057/diff/1/src/gpu/gl/GrGLProgram.cpp#newco...
src/gpu/gl/GrGLProgram.cpp:1620: // This block is replicated in
GrGLProgramStage::emitTextureLookup()
Truth in advertising. I'm surprised this hasn't given us a bug, but at the
moment I assume we aren't hitting this conditional *while* a customStage is
enabled? Need to either write a white-box test or keep a close eye on this going
forward.

[bsalomon, 2012-05-18 14:11:42]

On 2012/05/17 19:44:41, TomH wrote:
> http://codereview.appspot.com/6216057/diff/1/src/gpu/gl/GrGLProgram.cpp
> File src/gpu/gl/GrGLProgram.cpp (right):
> 
>
http://codereview.appspot.com/6216057/diff/1/src/gpu/gl/GrGLProgram.cpp#newco...
> src/gpu/gl/GrGLProgram.cpp:1300: GrGLShaderVar* radial2Params) {
> Effectively, fsCoordName was being used as an input, and sampleCoords as an
> output; now they both are in
> segments->fSampleCoords.
> 
>
http://codereview.appspot.com/6216057/diff/1/src/gpu/gl/GrGLProgram.cpp#newco...
> src/gpu/gl/GrGLProgram.cpp:1620: // This block is replicated in
> GrGLProgramStage::emitTextureLookup()
> Truth in advertising. I'm surprised this hasn't given us a bug, but at the
> moment I assume we aren't hitting this conditional *while* a customStage is
> enabled? Need to either write a white-box test or keep a close eye on this
going
> forward.

LGTM

[TomH, 2012-05-18 14:58:06]

Committed as r3996.