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/GrRenderTask.cpp

diff --git a/src/gpu/GrRenderTask.cpp b/src/gpu/GrRenderTask.cpp
index 89f9b5c..4397cf9 100644
--- a/src/gpu/GrRenderTask.cpp
+++ b/src/gpu/GrRenderTask.cpp
@@ -284,7 +284,7 @@
     fTargets.push_back(std::move(view));
 }
 
-#ifdef SK_DEBUG
+#if GR_TEST_UTILS
 void GrRenderTask::dump(bool printDependencies) const {
     SkDebugf("--------------------------------------------------------------\n");
     SkDebugf("%s - renderTaskID: %d\n", this->name(), fUniqueID);