Made isEqual in GrFragmentProcessor recursive

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 111 matching lines @@
     for (int i = 0; i < this->numTextures(); ++i) {
         if (this->textureAccess(i) != that.textureAccess(i)) {
             return false;
         }
     }
     return true;
 }
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
+bool GrFragmentProcessor::isEqual(const GrFragmentProcessor& that,
+                                  bool ignoreCoordTransforms) const {
+    if (this->classID() != that.classID() ||
+        !this->hasSameTextureAccesses(that)) {
+        return false;
+    }
+    if (ignoreCoordTransforms) {
+        if (this->numTransforms() != that.numTransforms()) {

[tomhudson, 2015/08/17 17:33:10]

Why does the number of transforms matter IF we're told to ignore them?

[joshualitt, 2015/08/17 17:38:42]

Its a bit of a misnomer, it actually just ignores the CoordTransform matrices.

+            return false;
+        }
+    } else if (!this->hasSameTransforms(that)) {
+        return false;
+    }
+    if (!this->onIsEqual(that)) {
+        return false;
+    }
+    if (this->numChildProcessors() != that.numChildProcessors()) {
+        return false;
+    }
+    for (int i = 0; i < this->numChildProcessors(); ++i) {
+        if (!this->childProcessor(i).isEqual(that.childProcessor(i), ignoreCoordTransforms)) {
+            return false;
+        }
+    }
+    return true;
+}
+
 void GrFragmentProcessor::addCoordTransform(const GrCoordTransform* transform) {
     fCoordTransforms.push_back(transform);
     fUsesLocalCoords = fUsesLocalCoords || transform->sourceCoords() == kLocal_GrCoordSet;
     SkDEBUGCODE(transform->setInProcessor();)
 }
 
 int GrFragmentProcessor::registerChildProcessor(const GrFragmentProcessor* child) {
     // Append the child's transforms to our transforms array and the child's textures array to our
     // textures array
     if (!child->fCoordTransforms.empty()) {
@@ skipping 41 matching lines @@
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
 // 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);
 }