Remove unused GrOpList::numOps()

This function was misleading because GrRenderTargetOpList::forwardCombine
can leave holes in the array which were still counted.

It would be harder to track this in op chaining redux.

Change-Id: Ib9619839012cb7dd2d0e4ccac31a231787f90f2d
Reviewed-on: https://skia-review.googlesource.com/c/166442
Commit-Queue: Brian Salomon <bsalomon@google.com>
Commit-Queue: Greg Daniel <egdaniel@google.com>
Auto-Submit: Brian Salomon <bsalomon@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>

include/private/GrOpList.h

@@ -86,7 +86,6 @@ public:
      */
     SkDEBUGCODE(virtual void dump(bool printDependencies) const;)
 
-    SkDEBUGCODE(virtual int numOps() const = 0;)
     SkDEBUGCODE(virtual int numClips() const { return 0; })
 
     // TODO: it would be nice for this to be hidden

src/gpu/GrRenderTargetOpList.h

@@ -118,8 +118,6 @@ public:
     GrRenderTargetOpList* asRenderTargetOpList() override { return this; }
 
     SkDEBUGCODE(void dump(bool printDependencies) const override;)
-
-    SkDEBUGCODE(int numOps() const override { return fRecordedOps.count(); })
     SkDEBUGCODE(int numClips() const override { return fNumClips; })
     SkDEBUGCODE(void visitProxies_debugOnly(const GrOp::VisitProxyFunc&) const;)
 

src/gpu/GrTextureOpList.h

@@ -58,8 +58,6 @@ public:
 
     SkDEBUGCODE(void dump(bool printDependencies) const override;)
 
-    SkDEBUGCODE(int numOps() const override { return fRecordedOps.count(); })
-
 private:
     void deleteOp(int index);
     void deleteOps();