Make 0-texture GrCustomEffects work

[bsalomon, 2012-07-30 19:29:10]

I couldn't push the changes as deep as I hoped. My next CL will allow
GrCustomStages to supply their own per-texture GrTextureParams objects. This
should make the change to allow multiple textures per stage a bit easier.

http://codereview.appspot.com/6448080/diff/4001/src/gpu/GrContext.cpp
File src/gpu/GrContext.cpp (right):

http://codereview.appspot.com/6448080/diff/4001/src/gpu/GrContext.cpp#newcode...
src/gpu/GrContext.cpp:1049: DrawCategory category = (DEFER_PATHS) ?
kBuffered_DrawCategory :
This part is somewhat orthogonal to the rest of the change. However, the fact
that the AutoStageDisable was inserted before the early-out to internalDrawPath
gave problems when I was inserting a dummy stage on every draw to test 0-texture
stages.

http://codereview.appspot.com/6448080/diff/4001/src/gpu/gl/GrGLProgramStage.h
File src/gpu/gl/GrGLProgramStage.h (left):

http://codereview.appspot.com/6448080/diff/4001/src/gpu/gl/GrGLProgramStage.h...
src/gpu/gl/GrGLProgramStage.h:40: // TODO: redundant with GrGLProgram.cpp
I noticed that this was still hanging around. It's no longer necessary with the
GrGLUniformManager (or GLUM as Tom likes to call it).

[TomH, 2012-07-31 15:13:56]

LGTM

http://codereview.appspot.com/6448080/diff/4001/src/gpu/GrContext.cpp
File src/gpu/GrContext.cpp (right):

http://codereview.appspot.com/6448080/diff/4001/src/gpu/GrContext.cpp#newcode...
src/gpu/GrContext.cpp:1033: // TODO: strokeWidth can't be larger than zero right
now.
If this comment is still true, could we assert it?

[bsalomon, 2012-07-31 17:56:07]

Closed with r4858.

http://codereview.appspot.com/6448080/diff/4001/src/gpu/GrContext.cpp
File src/gpu/GrContext.cpp (right):

http://codereview.appspot.com/6448080/diff/4001/src/gpu/GrContext.cpp#newcode...
src/gpu/GrContext.cpp:1049: DrawCategory category = (DEFER_PATHS) ?
kBuffered_DrawCategory :
On 2012/07/30 19:29:10, bsalomon wrote:
> This part is somewhat orthogonal to the rest of the change. However, the fact
> that the AutoStageDisable was inserted before the early-out to
internalDrawPath
> gave problems when I was inserting a dummy stage on every draw to test
0-texture
> stages.

Done.