Add support for transformedLocalCoords to GrDefaultGeoProc

src/gpu/GrGeometryProcessor.h

 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrGeometryProcessor_DEFINED
 #define GrGeometryProcessor_DEFINED
 
 #include "GrPrimitiveProcessor.h"
 
 /**
  * A GrGeometryProcessor is a flexible method for rendering a primitive.  The GrGeometryProcessor
  * has complete control over vertex attributes and uniforms(aside from the render target) but it
  * must obey the same contract as any GrPrimitiveProcessor, specifically it must emit a color and
  * coverage into the fragment shader.  Where this color and coverage come from is completely the
  * responsibility of the GrGeometryProcessor.
  */
 class GrGeometryProcessor : public GrPrimitiveProcessor {
 public:
     GrGeometryProcessor()
         : INHERITED(false)
         , fWillUseGeoShader(false)
-        , fHasLocalCoords(false) {}
+        , fLocalCoordsType(kIgnoredOrPosition_LocalCoordsType) {}
 
     bool willUseGeoShader() const override { return fWillUseGeoShader; }
 
     // TODO delete when paths are in batch
     void initBatchTracker(GrBatchTracker*, const GrPipelineInfo&) const override {}
 
     // TODO delete this when paths are in batch
     bool canMakeEqual(const GrBatchTracker& mine,
                       const GrPrimitiveProcessor& that,
                       const GrBatchTracker& theirs) const override {
         SkFAIL("Unsupported\n");
         return false;
     }
 
     // TODO Delete when paths are in batch
     void getInvariantOutputColor(GrInitInvariantOutput* out) const override {
         SkFAIL("Unsupported\n");
     }
     void getInvariantOutputCoverage(GrInitInvariantOutput* out) const override {
         SkFAIL("Unsupported\n");
     }
 
+    bool hasTransformedLocalCoords() const override {

[bsalomon, 2015/08/05 16:20:19]

enum getter here rather than two queries?

[joshualitt, 2015/08/05 16:32:30]

I'd rather just leave the queries.  All of this will go away when I circle back
to clean up program builder anyways, and I'd rather keep it as contained as
possible.  Unless you have a strong preference.

+        return kHasTransformed_LocalCoordsType == fLocalCoordsType;
+    }
+
+    bool hasExplicitLocalCoords() const override {
+        return kHasExplicit_LocalCoordsType == fLocalCoordsType;
+    }
+
 protected:
     /**
      * Subclasses call this from their constructor to register vertex attributes.  Attributes
      * will be padded to the nearest 4 bytes for performance reasons.
      * TODO After deferred geometry, we should do all of this inline in GenerateGeometry alongside
      * the struct used to actually populate the attributes.  This is all extremely fragile, vertex
      * attributes have to be added in the order they will appear in the struct which maps memory.
      * The processor key should reflect the vertex attributes, or there lack thereof in the
      * GrGeometryProcessor.
      */
     const Attribute& addVertexAttrib(const Attribute& attribute) {
         SkASSERT(fNumAttribs < kMaxVertexAttribs);
         fVertexStride += attribute.fOffset;
         fAttribs[fNumAttribs] = attribute;
         return fAttribs[fNumAttribs++];
     }
 
     void setWillUseGeoShader() { fWillUseGeoShader = true; }
 
-    // TODO hack see above
-    void setHasLocalCoords() { fHasLocalCoords = true; }
+    /**
+     * If a GrFragmentProcessor in the GrPipeline needs localCoods, we will provide them in one of
+     * three ways
+     * 1) LocalCoordTransform * Position - in Shader
+     * 2) LocalCoordTransform * ExplicitLocalCoords- in Shader
+     * 3) A transformation on the CPU uploaded via vertex attribute
+     * TODO make this GrBatches responsibility
+     */
+    enum LocalCoordsType {
+        kIgnoredOrPosition_LocalCoordsType,

[bsalomon, 2015/08/05 16:20:19]

Are there really ignored local local coords? Unused or not required? Maybe
separate values?

[joshualitt, 2015/08/05 16:32:30]

Acknowledged.

+        kHasExplicit_LocalCoordsType,
+        kHasTransformed_LocalCoordsType
+    };
+
+    void setHasExplicitLocalCoords() {
+        SkASSERT(kIgnoredOrPosition_LocalCoordsType == fLocalCoordsType);
+        fLocalCoordsType = kHasExplicit_LocalCoordsType;
+    }
+    void setHasTransformedLocalCoords() {
+        SkASSERT(kIgnoredOrPosition_LocalCoordsType == fLocalCoordsType);
+        fLocalCoordsType = kHasTransformed_LocalCoordsType;
+    }
 
 private:
-    bool hasExplicitLocalCoords() const override { return fHasLocalCoords; }
-
     bool fWillUseGeoShader;
-    bool fHasLocalCoords;
+    LocalCoordsType fLocalCoordsType;
 
     typedef GrPrimitiveProcessor INHERITED;
 };
 
 #endif