commit 4204800cd8f311f11491cf2eb7e32ca681f05489
author bsalomon <bsalomon@google.com> Thu Aug 27 16:43:48 2015 -0700
committer Commit bot <commit-bot@chromium.org> Thu Aug 27 16:43:48 2015 -0700
tree f2c836a3b44afcf98f871024155cff161e4e6039
parent 0153dea2f9c2fb23fb5e39933d21e11e162aac59

Convert child FPs from refs to pending executions when parent converts

Review URL: https://codereview.chromium.org/1315923004

src/gpu/GrProcessor.cpp

diff --git a/src/gpu/GrProcessor.cpp b/src/gpu/GrProcessor.cpp
index 3cbf777..7a3335c 100644
--- a/src/gpu/GrProcessor.cpp
+++ b/src/gpu/GrProcessor.cpp
@@ -131,8 +131,10 @@
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
 GrFragmentProcessor::~GrFragmentProcessor() {
+    // If we got here then our ref count must have reached zero, so we will have converted refs
+    // to pending executions for all children.
     for (int i = 0; i < fChildProcessors.count(); ++i) {
-        fChildProcessors[i]->unref();
+        fChildProcessors[i]->completedExecution();
     }
 }
 
@@ -218,6 +220,14 @@
     return index;
 }
 
+void GrFragmentProcessor::notifyRefCntIsZero() const {
+    // See comment above GrProgramElement for a detailed explanation of why we do this.
+    for (int i = 0; i < fChildProcessors.count(); ++i) {
+        fChildProcessors[i]->addPendingExecution();
+        fChildProcessors[i]->unref();
+    }
+}
+
 bool GrFragmentProcessor::hasSameTransforms(const GrFragmentProcessor& that) const {
     if (this->numTransforms() != that.numTransforms()) {
         return false;