commit 4133dc4e0b3a0bbabe9679441c773b4cd0d81261
author Robert Phillips <robertphillips@google.com> Wed Mar 11 15:55:55 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Mar 12 13:00:40 2020 +0000
tree 2f6c4ac0a1f10d785edabcd2ec0aab59d745693a
parent 9b5dba2ddba450639ceebdea107dca8186c6d567

Clean up programInfo creation in GrMeshDrawOp-derived Ops

This CL:
  Renames the existing createProgramInfo method to onCreateProgramInfo and makes it a virtual on GrMeshDrawOp

  Moves the non-virtual createProgramInfo (helper) calls to GrMeshDrawOp

  Changes onCreateProgramInfo to not return a ProgramInfo*. This is setting up to handle ops that create >1 programInfo (e.g., AAHairlineOp)

This CL leaves the following 8 ops in need of an onCreateProgramInfo implementation:

AAHairlineOp
AAFlatteningConvexPathOp
GrAtlasTextOp
GrShadowRRectOp
SmallPathOp
GrTextureOp
PrimitiveProcessorTest::Op
VertexColorSpaceBench::Op

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

src/gpu/ops/GrMeshDrawOp.cpp

diff --git a/src/gpu/ops/GrMeshDrawOp.cpp b/src/gpu/ops/GrMeshDrawOp.cpp
index a320c37..375ce55 100644
--- a/src/gpu/ops/GrMeshDrawOp.cpp
+++ b/src/gpu/ops/GrMeshDrawOp.cpp
@@ -15,6 +15,14 @@
 
 void GrMeshDrawOp::onPrepare(GrOpFlushState* state) { this->onPrepareDraws(state); }
 
+void GrMeshDrawOp::createProgramInfo(Target* target) {
+    this->createProgramInfo(&target->caps(),
+                            target->allocator(),
+                            target->outputView(),
+                            target->detachAppliedClip(),
+                            target->dstProxyView());
+}
+
 //////////////////////////////////////////////////////////////////////////////
 
 GrMeshDrawOp::PatternHelper::PatternHelper(Target* target, GrPrimitiveType primitiveType,