Initial change to create GeometryProcessor

src/gpu/gl/GrGLProgram.h

Index: src/gpu/gl/GrGLProgram.h
diff --git a/src/gpu/gl/GrGLProgram.h b/src/gpu/gl/GrGLProgram.h
index 5b9deb9f9178e46b85fc012c59a792b2e57b0909..0f89e07b0d734fbc452dac1ac7cefc54e3364168 100644
--- a/src/gpu/gl/GrGLProgram.h
+++ b/src/gpu/gl/GrGLProgram.h
@@ -41,6 +41,7 @@ public:
 
     static GrGLProgram* Create(GrGpuGL* gpu,
                                const GrGLProgramDesc& desc,
+                               const GrEffectStage* geometryProcessor,

[bsalomon, 2014/09/03 21:07:00]

should/can this be const& for the gp param (here and below)?

[joshua.litt, 2014/09/03 21:18:53]

GrGpuGL_program uses a NULL pointer to indicate no GP.  Derefing it to make a
const ref would be a bit awkward, but I can do it for setdata. 
GrGLProgramBuilder uses the NULL pointer to indicate no GP so it would still
require a pointer.

On 2014/09/03 21:07:00, bsalomon wrote:

[bsalomon, 2014/09/04 13:17:39]

Oh, of course. ptr is right.

                                const GrEffectStage* colorStages[],
                                const GrEffectStage* coverageStages[]);
 
@@ -158,6 +159,7 @@ public:
      */
     void setData(GrGpu::DrawType,
                  GrDrawState::BlendOptFlags,
+                 const GrEffectStage* geometryProcessor,
                  const GrEffectStage* colorStages[],
                  const GrEffectStage* coverageStages[],
                  const GrDeviceCoordTexture* dstCopy, // can be NULL
@@ -191,6 +193,7 @@ private:
     int                                 fDstCopyTexUnit;
 
     BuiltinUniformHandles               fBuiltinUniformHandles;
+    SkAutoTUnref<GrGLProgramEffects>    fGeometryProcessor;
     SkAutoTUnref<GrGLProgramEffects>    fColorEffects;
     SkAutoTUnref<GrGLProgramEffects>    fCoverageEffects;
     GrGLuint                            fProgramID;