diff options
| author |  Mike Klein <mtklein@chromium.org> | 2017-01-06 11:30:55 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-01-06 17:32:03 +0000 |
| commit | f55ea6a1deb21120944d406124a2984b5009260a (patch) | |
| tree | 7601c3e25a5ec9a35a59ea2cc8abec44095422da /bench/CoverageBench.cpp | |
| parent | c4bb2b9721adc8ef2453dcf5e20e8e6dfaf7be64 (diff) | |
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>
Diffstat (limited to 'bench/CoverageBench.cpp')
| -rw-r--r-- | bench/CoverageBench.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/bench/CoverageBench.cpp b/bench/CoverageBench.cpp index e1b53f879d..ce590e5c49 100644 --- a/bench/CoverageBench.cpp +++ b/bench/CoverageBench.cpp @@ -34,6 +34,11 @@ 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()); |