commit c17a3e9d68e54a05a4c47c957a52c4c938b10e0b
author Adlai Holler <adlai@google.com> Tue Apr 27 14:34:28 2021 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Apr 27 22:04:44 2021 +0000
tree e12977a0cb16af1b2ab74c06b539edf2e2b1e0e4
parent c4e9588621f7fb5d2325cea6417340d93004e6ee

Don't gather proxy intervals at all for NoOp ops tasks

Bug: skia:10877
Change-Id: Id2c4163e9346f4fd4f0fea2ab376d10821c49378
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/401857
Commit-Queue: Adlai Holler <adlai@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>

src/gpu/GrOpsTask.cpp

diff --git a/src/gpu/GrOpsTask.cpp b/src/gpu/GrOpsTask.cpp
index 552f8a2..862f151 100644
--- a/src/gpu/GrOpsTask.cpp
+++ b/src/gpu/GrOpsTask.cpp
@@ -881,6 +881,11 @@
 }
 
 void GrOpsTask::gatherProxyIntervals(GrResourceAllocator* alloc) const {
+    SkASSERT(this->isClosed());
+    if (this->isNoOp()) {
+        return;
+    }
+
     for (int i = 0; i < fDeferredProxies.count(); ++i) {
         SkASSERT(!fDeferredProxies[i]->isInstantiated());
         // We give all the deferred proxies a write usage at the very start of flushing. This