Revert "Add a new non-AA rect op that does not inherit from GrLegacyMeshDrawOp."
authorBrian Salomon <bsalomon@google.com>
Wed, 3 May 2017 20:58:59 +0000 (20:58 +0000)
committerSkia Commit-Bot <skia-commit-bot@chromium.org>
Wed, 3 May 2017 20:59:06 +0000 (20:59 +0000)
commit0f353327968530506dd3dd15fca79ef59fe013f1
treee225162be1818de922b243f184a03ce8a9060479
parent617824cc2bf47017e3cfc1251bea2296f2117db5
Revert "Add a new non-AA rect op that does not inherit from GrLegacyMeshDrawOp."

This reverts commit ff574e0eb79b83c2e797dec8f1661378876202d8.

Reason for revert: needs a merge

Original change's description:
> Add a new non-AA rect op that does not inherit from GrLegacyMeshDrawOp.
>
> This uses a new helper class, GrSimpleMeshDrawOpHelper, which it uses to fullfill the GrMeshDrawOp contract and to construct its GrPipline when flushed. The helper is intended to be used such that the op only stores a GrProcessorSet if it is constructed with a "nontrivial" GrPaint. "Trivial" currently means no fragment processors and src-over blending. The helper allows the op subclass to specify whether it supports stenciling via a template parameter. The helper class is initially intended to be used for ops that don't have per-vertex colors and construct a single GrPipeline at flush time, though perhaps this can be relaxed in future changes.
>
> On the microbenchmark "rotated_rects_bw_same_transparent_srcover" this produces a 18-20% reduction in time on my Z840 running Linux and 33% on my 2010 MacPro.
>
> Bug: skia:
> Change-Id: I9f655827a70bee585b0b0e1255371ffd995a0b80
> Reviewed-on: https://skia-review.googlesource.com/14604
> Commit-Queue: Brian Salomon <bsalomon@google.com>
> Reviewed-by: Brian Osman <brianosman@google.com>
>

TBR=bsalomon@google.com,robertphillips@google.com,brianosman@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Change-Id: I2893d6ff7c183a18f7d0ba82818701b80b681eb0
Reviewed-on: https://skia-review.googlesource.com/15280
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
12 files changed:
gn/gpu.gni
src/gpu/GrOpFlushState.h
src/gpu/GrPaint.cpp
src/gpu/GrPaint.h
src/gpu/GrPipeline.h
src/gpu/GrProcessorSet.cpp
src/gpu/GrProcessorSet.h
src/gpu/GrRenderTargetContext.cpp
src/gpu/ops/GrMeshDrawOp.cpp
src/gpu/ops/GrNewNonAAFillRectOp.cpp [deleted file]
src/gpu/ops/GrNewNonAAFillRectOp.h [deleted file]
src/gpu/ops/GrSimpleMeshDrawOpHelper.h [deleted file]