diff options
author | Florin Malita <fmalita@chromium.org> | 2017-01-27 12:05:27 -0500 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-01-27 17:48:08 +0000 |
commit | 00dca8cb09aaeca067cac022e97c4af31e656b37 (patch) | |
tree | b70e85179fae8f5c1815131c8fbf8e8e5bae5715 /src/core/SkBlitter.h | |
parent | a6f6f77566f210788f83a531067f1b5222d2d33a (diff) |
Remove SkBlitter:resetShaderContext()
Refactor SkDraw::drawVertices() to reset the whole blitter instead.
No noticeable bench diff.
out/Release/nanobench --config 8888 -m patch_grid_colors_tex
Before:
9/9 MB 1 9.24ms 11.4ms 11.5ms 15ms 16% █▆▅▄▄▃▂▂▁▁ 8888 patch_grid_colors_texs_big
9/9 MB 1 5.2ms 5.2ms 5.2ms 5.21ms 0% █▃▂▁▂▂▃▂▂▁ 8888 patch_grid_colors_texs_medium
9/9 MB 2 378µs 380µs 380µs 382µs 0% █▆▅▄▄▁▄▄▆▄ 8888 patch_grid_colors_texs_small
After:
9/9 MB 1 9.29ms 11.5ms 11.6ms 15.3ms 17% █▇▅▄▄▃▂▂▁▁ 8888 patch_grid_colors_texs_big
9/9 MB 1 5.15ms 5.16ms 5.16ms 5.18ms 0% █▄▁▁▄▂▁▂▃▂ 8888 patch_grid_colors_texs_medium
9/9 MB 2 378µs 380µs 380µs 382µs 0% █▅▄▃▁▄▄▄▃▄ 8888 patch_grid_colors_texs_small
R=reed@google.com,mtklein@google.com,herb@google.com
BUG=skia:6168
Change-Id: I0b6b7d1297323681d8d2838040450c4e1afabc17
Reviewed-on: https://skia-review.googlesource.com/7626
Reviewed-by: Mike Klein <mtklein@chromium.org>
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Florin Malita <fmalita@chromium.org>
Diffstat (limited to 'src/core/SkBlitter.h')
-rw-r--r-- | src/core/SkBlitter.h | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/core/SkBlitter.h b/src/core/SkBlitter.h index ac4712ed48..cab2afc974 100644 --- a/src/core/SkBlitter.h +++ b/src/core/SkBlitter.h @@ -108,11 +108,6 @@ public: virtual bool isNullBlitter() const; /** - * Special methods for SkShaderBlitter. On all other classes this is a no-op. - */ - virtual bool resetShaderContext(const SkShader::ContextRec&); - - /** * Special methods for blitters that can blit more than one row at a time. * This function returns the number of rows that this blitter could optimally * process at a time. It is still required to support blitting one scanline |