commit 1e0136ecdb12940bae0ec3284c28e532611dc4dd
author John Stiles <johnstiles@google.com> Wed Aug 12 18:44:00 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Aug 13 13:05:09 2020 +0000
tree 3f556642b547e2a08d65cf208c747d9b016e9f3c
parent df9ed89229db82e606c67fbf0f5c24b4c383c1e1

Remove more dump-info utility methods when GR_TEST_UTILS is off.

`GrRenderTask::dump` is now enabled by GR_TEST_UTILS, not SK_DEBUG.
`GrOpsTask::dump` is now enabled by GR_TEST_UTILS, not SK_DEBUG.
`GrOp::dumpInfo` no longer exists when GR_TEST_UTILS is off.

Various GrRenderTask subclasses' `name` methods are now enabled by
GR_TEST_UTILS, not SK_DEBUG. (`name` is only referenced by `dump`.)

Note that GR_TEST_UTILS can be enabled with SK_DEBUG turned off, which
makes a lot of these ripple-effect changes unavoidable.

Change-Id: Ie0fafed6f05c7d4083e769cd2f058db9ecc01d5d
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/309882
Auto-Submit: John Stiles <johnstiles@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: John Stiles <johnstiles@google.com>

src/gpu/GrTransferFromRenderTask.h

diff --git a/src/gpu/GrTransferFromRenderTask.h b/src/gpu/GrTransferFromRenderTask.h
index 46ac253..aa80be4 100644
--- a/src/gpu/GrTransferFromRenderTask.h
+++ b/src/gpu/GrTransferFromRenderTask.h
@@ -41,8 +41,10 @@
 
     bool onExecute(GrOpFlushState*) override;
 
-#ifdef SK_DEBUG
+#if GR_TEST_UTILS
     const char* name() const final { return "TransferFrom"; }
+#endif
+#ifdef SK_DEBUG
     void visitProxies_debugOnly(const GrOp::VisitProxyFunc& fn) const override {
         fn(fSrcProxy.get(), GrMipmapped::kNo);
     }