commit 02dd0ed8ce5e4436a4497f9f6cd970bac5e8b380
author Robert Phillips <robertphillips@google.com> Mon Nov 09 17:03:34 2020 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Nov 10 13:37:46 2020 +0000
tree f3331117dbc700744438c639bd352bbe8947aae0
parent 5b8598952931fc334c6af6c7bc7e8391928abe14

Change UnrefDDLTask to just be the DDLTask

Change-Id: Ib5ff9f3f3d9cbc8860e3a2ba74e6da4355197662
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/332600
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>

src/gpu/GrRenderTask.cpp

diff --git a/src/gpu/GrRenderTask.cpp b/src/gpu/GrRenderTask.cpp
index d63aa2e..a68910e 100644
--- a/src/gpu/GrRenderTask.cpp
+++ b/src/gpu/GrRenderTask.cpp
@@ -257,19 +257,16 @@
 }
 
 bool GrRenderTask::isInstantiated() const {
-    // Some renderTasks (e.g. GrTransferFromRenderTask) don't have any targets.
-    if (0 == this->numTargets()) {
-        return true;
-    }
-    GrSurfaceProxy* proxy = this->target(0).proxy();
+    for (const GrSurfaceProxyView& target : fTargets) {
+        GrSurfaceProxy* proxy = target.proxy();
+        if (!proxy->isInstantiated()) {
+            return false;
+        }
 
-    if (!proxy->isInstantiated()) {
-        return false;
-    }
-
-    GrSurface* surface = proxy->peekSurface();
-    if (surface->wasDestroyed()) {
-        return false;
+        GrSurface* surface = proxy->peekSurface();
+        if (surface->wasDestroyed()) {
+            return false;
+        }
     }
 
     return true;
@@ -291,10 +288,9 @@
 
     if (!fTargets.empty()) {
         SkDebugf("Targets: \n");
-        for (int i = 0; i < fTargets.count(); ++i) {
-            GrSurfaceProxy* proxy = fTargets[i].proxy();
-            SkDebugf("[%d]: proxyID: %d - surfaceID: %d\n",
-                     i,
+        for (const GrSurfaceProxyView& target : fTargets) {
+            GrSurfaceProxy* proxy = target.proxy();
+            SkDebugf("proxyID: %d - surfaceID: %d\n",
                      proxy ? proxy->uniqueID().asUInt() : -1,
                      proxy && proxy->peekSurface()
                             ? proxy->peekSurface()->uniqueID().asUInt()