Move willUseInputColor check to computeInvariantOutput

include/gpu/GrProcessor.h

 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrProcessor_DEFINED
 #define GrProcessor_DEFINED
 
@@ skipping 21 matching lines @@
     effect use the macro GR_CREATE_STATIC_EFFECT declared below.
   */
 class GrProcessor : public GrProgramElement {
 public:
     SK_DECLARE_INST_COUNT(GrProcessor)
 
     virtual ~GrProcessor();
 
     struct InvariantOutput{
         InvariantOutput() : fColor(0), fValidFlags(0), fIsSingleComponent(false),
-                            fInputIsOnlyModulated(true) {}
+                            fInputIsOnlyModulated(true), fWillUseInputColor(true) {}
 
         void mulByUnknownOpaqueColor() {
             if (this->isOpaque()) {
                 fValidFlags = kA_GrColorComponentFlag;
                 fIsSingleComponent = false;
             } else {
                 // Since the current state is not opaque we no longer care if the color being
                 // multiplied is opaque.
                 this->mulByUnknownColor(); 
             }
@@ skipping 31 matching lines @@
             fColor = color;
             fIsSingleComponent = false;
             fInputIsOnlyModulated = false;
         }
 
         void setToUnknown() {
             this->setToUnknownPriv();
             fInputIsOnlyModulated = false;
         }
 
+        /**
+         * If the effect will generate a result that does not depend on the input color value then
+         * it must call this function from its onComputeInvariantOutput(). Otherwise, when its
+         * generated backend-specific code might fail during variable binding due to unused
+         * variables.
+         */
+        void setWillNotUseInputColor() {

[bsalomon, 2014/10/13 20:54:42]

Instead of a new setter can we make it be a required param to all the helper
setters where it can't be inferred (e.g. setUnknown)?

[egdaniel, 2014/10/14 17:49:57]

Done.

+            fWillUseInputColor = false;
+        }
+
         bool isOpaque() const {
             return ((fValidFlags & kA_GrColorComponentFlag) && 0xFF == GrColorUnpackA(fColor));
         }
 
         bool isSolidWhite() const {
             return (fValidFlags == kRGBA_GrColorComponentFlags && 0xFFFFFFFF == fColor);
         }
 
         GrColor color() const { return fColor; }
         uint8_t validFlags() const { return fValidFlags; }
@@ skipping 25 matching lines @@
          * If alpha is valid, check that any valid R,G,B values are <= A
          */
         SkDEBUGCODE(bool validPreMulColor() const;)
 
         // Friended class that have "controller" code which loop over stages calling
         // computeInvarianteOutput(). These controllers may need to manually adjust the internal
         // members of InvariantOutput
         friend class GrDrawState;
         friend class GrOptDrawState;
         friend class GrPaint;
+        friend class GrProcessor;
 
         GrColor fColor;
         uint32_t fValidFlags;
         bool fIsSingleComponent;
         bool fInputIsOnlyModulated;
+        bool fWillUseInputColor;
     };
 
     /**
      * This function is used to perform optimizations. When called the invarientOuput param
      * indicate whether the input components to this effect in the FS will have known values.
      * In inout the validFlags member is a bitfield of GrColorComponentFlags. The isSingleComponent
      * member indicates whether the input will be 1 or 4 bytes. The function updates the members of
      * inout to indicate known values of its output. A component of the color member only has
      * meaning if the corresponding bit in validFlags is set.
      */
     void computeInvariantOutput(InvariantOutput* inout) const {
+        inout->fWillUseInputColor = true;
         this->onComputeInvariantOutput(inout);
 #ifdef SK_DEBUG
         inout->validate();
 #endif
     }
 
     /** This object, besides creating back-end-specific helper objects, is used for run-time-type-
         identification. The factory should be an instance of templated class,
         GrTBackendEffectFactory. It is templated on the subclass of GrProcessor. The subclass must
         have a nested type (or typedef) named GLProcessor which will be the subclass of
@@ skipping 115 matching lines @@
     SkSTArray<4, const GrTextureAccess*, true>   fTextureAccesses;
     bool                                         fWillReadFragmentPosition;
 
     typedef GrProgramElement INHERITED;
 };
 
 class GrFragmentProcessor : public GrProcessor {
 public:
     GrFragmentProcessor()
         : INHERITED()
-        , fWillReadDstColor(false)
-        , fWillUseInputColor(true) {}
+        , fWillReadDstColor(false) {}
 
     virtual const GrBackendFragmentProcessorFactory& getFactory() const = 0;
 
     /** Will this effect read the destination pixel value? */
     bool willReadDstColor() const { return fWillReadDstColor; }
 
-    /** Will this effect read the source color value? */
-    bool willUseInputColor() const { return fWillUseInputColor; }
-
 protected:
     /**
      * If the effect subclass will read the destination pixel value then it must call this function
      * from its constructor. Otherwise, when its generated backend-specific effect class attempts
      * to generate code that reads the destination pixel it will fail.
      */
     void setWillReadDstColor() { fWillReadDstColor = true; }
 
-    /**
-     * If the effect will generate a result that does not depend on the input color value then it
-     * must call this function from its constructor. Otherwise, when its generated backend-specific
-     * code might fail during variable binding due to unused variables.
-     */
-    void setWillNotUseInputColor() { fWillUseInputColor = false; }
-
 private:
     bool                                         fWillReadDstColor;
-    bool                                         fWillUseInputColor;
 
     typedef GrProcessor INHERITED;
 };
 
 /**
  * This creates an effect outside of the effect memory pool. The effect's destructor will be called
  * at global destruction time. NAME will be the name of the created GrProcessor.
  */
 #define GR_CREATE_STATIC_FRAGMENT_PROCESSOR(NAME, EFFECT_CLASS, ARGS)                             \
 static SkAlignedSStorage<sizeof(EFFECT_CLASS)> g_##NAME##_Storage;                                \
 static GrFragmentProcessor*                                                                       \
 NAME SkNEW_PLACEMENT_ARGS(g_##NAME##_Storage.get(), EFFECT_CLASS, ARGS);                          \
 static SkAutoTDestroy<GrFragmentProcessor> NAME##_ad(NAME);
 
 #endif