Fix few inconsistencies in handling fixed-function texture coordinate set generation

src/gpu/gl/GrGLProgram.cpp

Index: src/gpu/gl/GrGLProgram.cpp
diff --git a/src/gpu/gl/GrGLProgram.cpp b/src/gpu/gl/GrGLProgram.cpp
index 7ddd8613f19b571f606ac6000682bb4fc1b8c3e9..7776f499dee1c0e00bb0954ff0670dfe3cdc40f2 100644
--- a/src/gpu/gl/GrGLProgram.cpp
+++ b/src/gpu/gl/GrGLProgram.cpp
@@ -241,8 +241,8 @@ void GrGLProgram::setData(GrDrawState::BlendOptFlags blendOpts,
     fColorEffects->setData(fGpu, fUniformManager, colorStages);
     fCoverageEffects->setData(fGpu, fUniformManager, coverageStages);
 
-    if (!fHasVertexShader) {
-        fGpu->disableUnusedTexGen(fNumTexCoordSets);
+    if (fGpu->glCaps().fixedFunctionSupport()) {

[Chris Dalton, 2013/10/31 17:49:09]

I think it would be best to leave this as 'if (!fHasVertexShader)'.

TexGen state applies to the the fixed function vertex shader, so when you have
you own shader, it's ignored. We could potentially save some work by just
leaving it if there's a vertex shader.

[Kimmo Kinnunen, 2013/11/01 06:32:11]

Done.

+        fGpu->flushTexGenSettings(fNumTexCoordSets);
     }
 }