Add render task type to flush-time op spew

In the flush-time op spew it is clear which renderTasks are the Ops-based ones. This clarifies which of the other varieties occur.

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

Author: rphilli

src/gpu/GrCopyRenderTask.h

@@ -40,6 +40,7 @@ class GrCopyRenderTask final : public GrRenderTask {
     bool onExecute(GrOpFlushState*) override;
 
 #ifdef SK_DEBUG
+    const char* name() const final { return "Copy"; }
     void visitProxies_debugOnly(const GrOp::VisitProxyFunc& fn) const override {
         fn(fSrcView.proxy(), GrMipMapped::kNo);
     }

src/gpu/GrOpsTask.h

@@ -110,9 +110,12 @@ class GrOpsTask : public GrRenderTask {
 
     void discard();
 
-    SkDEBUGCODE(void dump(bool printDependencies) const override;)
-    SkDEBUGCODE(int numClips() const override { return fNumClips; })
-    SkDEBUGCODE(void visitProxies_debugOnly(const GrOp::VisitProxyFunc&) const override;)
+#ifdef SK_DEBUG
+    void dump(bool printDependencies) const override;
+    const char* name() const final { return "Ops"; }
+    int numClips() const override { return fNumClips; }
+    void visitProxies_debugOnly(const GrOp::VisitProxyFunc&) const override;
+#endif
 
 #if GR_TEST_UTILS
     int numOpChains() const { return fOpChains.count(); }

src/gpu/GrRenderTask.cpp

@@ -272,7 +272,8 @@ bool GrRenderTask::isInstantiated() const {
 void GrRenderTask::dump(bool printDependencies) const {
     SkDebugf("--------------------------------------------------------------\n");
     GrSurfaceProxy* proxy = fTargetView.proxy();
-    SkDebugf("renderTaskID: %d - proxyID: %d - surfaceID: %d\n", fUniqueID,
+    SkDebugf("%s - renderTaskID: %d - proxyID: %d - surfaceID: %d\n",
+             this->name(), fUniqueID,
              proxy ? proxy->uniqueID().asUInt() : -1,
              proxy && proxy->peekSurface()
                      ? proxy->peekSurface()->uniqueID().asUInt()

src/gpu/GrRenderTask.h

@@ -73,6 +73,7 @@ class GrRenderTask : public SkRefCnt {
      * Dump out the GrRenderTask dependency DAG
      */
     virtual void dump(bool printDependencies) const;
+    virtual const char* name() const = 0;
 
     virtual int numClips() const { return 0; }
 

src/gpu/GrTextureResolveRenderTask.h

@@ -35,7 +35,8 @@ class GrTextureResolveRenderTask final : public GrRenderTask {
     bool onExecute(GrOpFlushState*) override;
 
 #ifdef SK_DEBUG
-    SkDEBUGCODE(void visitProxies_debugOnly(const GrOp::VisitProxyFunc&) const override;)
+    const char* name() const final { return "TextureResolve"; }
+    void visitProxies_debugOnly(const GrOp::VisitProxyFunc&) const override;
 #endif
 
     struct Resolve {

src/gpu/GrTransferFromRenderTask.h

@@ -42,6 +42,7 @@ class GrTransferFromRenderTask final : public GrRenderTask {
     bool onExecute(GrOpFlushState*) override;
 
 #ifdef SK_DEBUG
+    const char* name() const final { return "TransferFrom"; }
     void visitProxies_debugOnly(const GrOp::VisitProxyFunc& fn) const override {
         fn(fSrcProxy.get(), GrMipMapped::kNo);
     }

src/gpu/GrWaitRenderTask.h

@@ -36,6 +36,7 @@ class GrWaitRenderTask final : public GrRenderTask {
     bool onExecute(GrOpFlushState*) override;
 
 #ifdef SK_DEBUG
+    const char* name() const final { return "Wait"; }
     // No non-dst proxies.
     void visitProxies_debugOnly(const GrOp::VisitProxyFunc& fn) const override {}
 #endif