All child GrFragmentProcs' transforms and textures will be stored in the root GrFragmentProc in pre…

src/gpu/GrProcessor.cpp

 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "GrProcessor.h"
 #include "GrContext.h"
 #include "GrCoordTransform.h"
@@ skipping 117 matching lines @@
 }
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
 void GrFragmentProcessor::addCoordTransform(const GrCoordTransform* transform) {
     fCoordTransforms.push_back(transform);
     fUsesLocalCoords = fUsesLocalCoords || transform->sourceCoords() == kLocal_GrCoordSet;
     SkDEBUGCODE(transform->setInProcessor();)
 }
 
-void GrFragmentProcessor::registerChildProcessor(GrFragmentProcessor* child) {
-    fChildProcessors.push_back(child);
+int GrFragmentProcessor::registerChildProcessor(GrFragmentProcessor* child) {
+    // Append the child's transforms to our transforms array and the child's textures array to our
+    // textures array, and empty the child's arrays.
+    if (!child->fCoordTransforms.empty()) {
+        fCoordTransforms.push_back_n(child->fCoordTransforms.count(),
+                                     child->fCoordTransforms.begin());
+        child->fCoordTransforms.reset();

[joshualitt, 2015/08/12 14:49:52]

I think we don't want to reset the child's data because the child may be used
elsewhere.  I think registerChildProcessor might have to take a const
GrFragmentProcessor*.

However, double storing the pointers in both the child and the parent doesn't
seem like a big deal to me because we shouldn't have very many coord transforms
or textureaccesses.  Thoughts Brian?

[wangyix, 2015/08/12 15:09:24]

Done.

[bsalomon, 2015/08/12 15:14:22]

sgtm

+    }
+    if (!child->fTextureAccesses.empty()) {
+        fTextureAccesses.push_back_n(child->fTextureAccesses.count(),
+                                     child->fTextureAccesses.begin());
+        child->fTextureAccesses.reset();
+    }
+
+    int index = fChildProcessors.count();
+    fChildProcessors.push_back(GrFragmentStage(child));
+
+    SkDEBUGCODE(child->fIsChild = true);
+
+    if (child->willReadFragmentPosition())
+        this->setWillReadFragmentPosition();
+
+    return index;
 }
 
 bool GrFragmentProcessor::hasSameTransforms(const GrFragmentProcessor& that) const {
-    if (fCoordTransforms.count() != that.fCoordTransforms.count()) {
+    if (this->numTransforms() != that.numTransforms()) {
         return false;
     }
-    int count = fCoordTransforms.count();
+    int count = this->numTransforms();
     for (int i = 0; i < count; ++i) {
-        if (*fCoordTransforms[i] != *that.fCoordTransforms[i]) {
+        if (this->coordTransform(i) != that.coordTransform(i)) {
             return false;
         }
     }
     return true;
 }
 
 void GrFragmentProcessor::computeInvariantOutput(GrInvariantOutput* inout) const {
     this->onComputeInvariantOutput(inout);
 }
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
 // Initial static variable from GrXPFactory
 int32_t GrXPFactory::gCurrXPFClassID =
         GrXPFactory::kIllegalXPFClassID;
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
 // GrProcessorDataManager lives in the same pool
 void* GrProcessorDataManager::operator new(size_t size) {
     return MemoryPoolAccessor().pool()->allocate(size);
 }
 
 void GrProcessorDataManager::operator delete(void* target) {
     return MemoryPoolAccessor().pool()->release(target);
 }