Added class AutoFragmentChildProcAdvance to be constructed before a child emitCode and destructed a…

src/gpu/gl/builders/GrGLFragmentShaderBuilder.h

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrGLFragmentShaderBuilder_DEFINED
 #define GrGLFragmentShaderBuilder_DEFINED
 
@@ skipping 59 matching lines @@
 
     SkString getMangleStringThisLevel() const {
         SkString ret;
         int childNumber = this->getChildNumberThisLevel();
         if (childNumber >= 0) {
             ret.printf("_c%d", childNumber);
         }
         return ret;
     }
 
+    // This class is like AutoStageAdvance but used for the child procs of a fragment proc.

[tomhudson, 2015/08/14 15:58:48]

This comment requires the reader to understand AutoStageAdvance, which isn't
even in the same source file. Try to make it self-contained? The long list of
arguments sure makes it look like something complex is going on.

[wangyix, 2015/08/14 16:07:27]

Acknowledged.

+    class AutoFragmentChildProcAdvance {
+        typedef GrGLProcessor::TransformedCoordsArray TransformedCoordsArray;
+        typedef GrGLProcessor::TextureSamplerArray TextureSamplerArray;
+    public:
+        AutoFragmentChildProcAdvance(int childProcIndex,
+                                     GrGLFPBuilder* builder,
+                                     const GrFragmentProcessor& fp,
+                                     const char* outputColor,
+                                     const TransformedCoordsArray& coords,
+                                     const TextureSamplerArray& samplers,
+                                     const GrFragmentProcessor** childFp,
+                                     SkString* childOutputColor,
+                                     TransformedCoordsArray* childCoords,
+                                     TextureSamplerArray* childSamplers);
+
+        ~AutoFragmentChildProcAdvance();
+    private:
+        GrGLFragmentBuilder* fFsb;
+    };
+
 private:
     /*
      * State that tracks which child proc in the proc tree is currently emitting code.  This is
      * used to update the fMangleString, which is used to mangle the names of uniforms and functions
      * emitted by the proc.  fSubstageIndices is a stack: its count indicates how many levels deep
      * we are in the tree, and its second-to-last value is the index of the child proc at that
      * level which is currently emitting code. For example, if fSubstageIndices = [3, 1, 2, 0], that
      * means we're currently emitting code for the base proc's 3rd child's 1st child's 2nd child.
      */
     SkTArray<int> fSubstageIndices;
@@ skipping 122 matching lines @@
     // the program creator
     bool fHasReadDstColor;
     bool fHasReadFragmentPosition;
 
     friend class GrGLProgramBuilder;
 
     typedef GrGLXPFragmentBuilder INHERITED;
 };
 
 #endif