diff options
| author |  Mike Klein <mtklein@chromium.org> | 2017-05-09 14:57:58 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-05-09 20:04:44 +0000 |
| commit | fdf31035a9639c2d377bdf2ddb404d269f8ca42a (patch) | |
| tree | 1243f154d371f25971833443e24c2136728c3340 /src/effects/SkLumaColorFilter.cpp | |
| parent | b8fbab2a1c79764e490e65ebc80288cdc0494237 (diff) | |
void SkColorFilter::onAppendStages(...)
SkColorFilter::onAppendStages() can't actually fail today.
Let's enforce that. This means the fallback is now used
only for color filters that have not implemented onAppendStages().
Change-Id: Ica3939685694f6186727766b54914b9ba05ca68c
Reviewed-on: https://skia-review.googlesource.com/16231
Reviewed-by: Matt Sarett <msarett@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src/effects/SkLumaColorFilter.cpp')
| -rw-r--r-- | src/effects/SkLumaColorFilter.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/effects/SkLumaColorFilter.cpp b/src/effects/SkLumaColorFilter.cpp index b2f6b47a69..f0e58c06f8 100644 --- a/src/effects/SkLumaColorFilter.cpp +++ b/src/effects/SkLumaColorFilter.cpp @@ -55,12 +55,11 @@ void SkLumaColorFilter::filterSpan4f(const SkPM4f src[], int count, SkPM4f dst[] } } -bool SkLumaColorFilter::onAppendStages(SkRasterPipeline* p, +void SkLumaColorFilter::onAppendStages(SkRasterPipeline* p, SkColorSpace* dst, SkArenaAlloc* scratch, bool shaderIsOpaque) const { p->append(SkRasterPipeline::luminance_to_alpha); - return true; } sk_sp<SkColorFilter> SkLumaColorFilter::Make() { |