diff options
| author |  Mike Klein <mtklein@chromium.org> | 2017-01-07 22:04:51 +0000 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-01-07 22:05:05 +0000 |
| commit | 86d55b312a2649d80890ccf75f24571ada0265f1 (patch) | |
| tree | 8e25adaa99e9dfb8f7d0f88cc4c00099b7b04e8a /bench | |
| parent | c3063e54cb1274359f36d9720d4c1b3acf37c203 (diff) | |
Revert "Retry "SkRasterPipelineBlitter: support A8"..."
This reverts commit f55ea6a1deb21120944d406124a2984b5009260a.
Reason for revert: crbug.com/679147
Original change's description:
> Retry "SkRasterPipelineBlitter: support A8"...
>
> ...preferring SkA8_Coverage_Blitter over SkRasterPipelineBlitter.
>
> I think we could make this work with SkRasterPipelineBlitter (tell it, draw white in Src mode with this mask), but the existing blitter is pretty hard to beat in efficiency and correctness.
>
> CQ_INCLUDE_TRYBOTS=skia.primary:Perf-Ubuntu-Clang-GCE-CPU-AVX2-x86_64-Debug-MSAN,Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD
>
>
> Change-Id: I72df9995c63f3334d8111c59711818cb5ed1e63c
> Reviewed-on: https://skia-review.googlesource.com/6627
> Reviewed-by: Mike Klein <mtklein@chromium.org>
> Commit-Queue: Mike Klein <mtklein@chromium.org>
>
TBR=mtklein@chromium.org,brianosman@google.com,reed@google.com,reviews@skia.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: I6a36b4c087a52e54f4d591ded40e6a202fb77068
Reviewed-on: https://skia-review.googlesource.com/6760
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'bench')
| -rw-r--r-- | bench/CoverageBench.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/bench/CoverageBench.cpp b/bench/CoverageBench.cpp index ce590e5c49..e1b53f879d 100644 --- a/bench/CoverageBench.cpp +++ b/bench/CoverageBench.cpp @@ -34,11 +34,6 @@ public: fPath.quadTo(250, 0, 0, 500); fPixmap.alloc(SkImageInfo::MakeA8(500, 500)); - if (!drawCoverage) { - // drawPathCoverage() goes out of its way to work fine with an uninitialized - // dst buffer, even in "SrcOver" mode, but ordinary drawing sure doesn't. - fPixmap.erase(0); - } fIdentity.setIdentity(); fRC.setRect(fPath.getBounds().round()); |