commit e25c30034a633d873a00d89cee4451545941a7ef
author Herb Derby <herb@google.com> Tue Oct 27 15:57:27 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Oct 28 19:10:01 2020 +0000
tree 0e1e8042e1d9453b58ecbb324b92bbd1f85637dd
parent 541601602ee94b1fbf04d9b2fe0661951960f97c

use SkArenaAlloc for combineIfPossible

Now that GrOps no longer needs a GrMemoryPool to be
deleted, just pass down the the SkArenaAlloc instead
of both the GrMemoryPool and SkArenaAlloc.

The alloc is only used for two ops, but we pass it
to all the ops most of which don't use it.

Change-Id: I873efcdfe44b446f2ac5089ea8425dff257e318c
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/330118
Reviewed-by: Michael Ludwig <michaelludwig@google.com>
Commit-Queue: Herb Derby <herb@google.com>

src/gpu/ops/GrFillRRectOp.cpp

diff --git a/src/gpu/ops/GrFillRRectOp.cpp b/src/gpu/ops/GrFillRRectOp.cpp
index b55e679..0a78811 100644
--- a/src/gpu/ops/GrFillRRectOp.cpp
+++ b/src/gpu/ops/GrFillRRectOp.cpp
@@ -43,7 +43,7 @@
 
     GrProcessorSet::Analysis finalize(const GrCaps&, const GrAppliedClip*,
                                       bool hasMixedSampledCoverage, GrClampType) final;
-    CombineResult onCombineIfPossible(GrOp*, GrRecordingContext::Arenas*, const GrCaps&) final;
+    CombineResult onCombineIfPossible(GrOp*, SkArenaAlloc*, const GrCaps&) final;
 
     void visitProxies(const VisitProxyFunc& fn) const override {
         if (fProgramInfo) {
@@ -278,9 +278,7 @@
     return analysis;
 }
 
-GrDrawOp::CombineResult FillRRectOp::onCombineIfPossible(GrOp* op,
-                                                         GrRecordingContext::Arenas*,
-                                                         const GrCaps& caps) {
+GrOp::CombineResult FillRRectOp::onCombineIfPossible(GrOp* op, SkArenaAlloc*, const GrCaps& caps) {
     const auto& that = *op->cast<FillRRectOp>();
     if (!fHelper.isCompatible(that.fHelper, caps, this->bounds(), that.bounds())) {
         return CombineResult::kCannotCombine;