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/core/SkColorMatrixFilterRowMajor255.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/core/SkColorMatrixFilterRowMajor255.cpp')
| -rw-r--r-- | src/core/SkColorMatrixFilterRowMajor255.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/core/SkColorMatrixFilterRowMajor255.cpp b/src/core/SkColorMatrixFilterRowMajor255.cpp index f3dc41f0fe..bd88c927d1 100644 --- a/src/core/SkColorMatrixFilterRowMajor255.cpp +++ b/src/core/SkColorMatrixFilterRowMajor255.cpp @@ -231,7 +231,7 @@ static void set_concat(SkScalar result[20], const SkScalar outer[20], const SkSc // End duplication ////// -bool SkColorMatrixFilterRowMajor255::onAppendStages(SkRasterPipeline* p, +void SkColorMatrixFilterRowMajor255::onAppendStages(SkRasterPipeline* p, SkColorSpace* dst, SkArenaAlloc* scratch, bool shaderIsOpaque) const { @@ -254,7 +254,6 @@ bool SkColorMatrixFilterRowMajor255::onAppendStages(SkRasterPipeline* p, if (!willStayOpaque) { p->append(SkRasterPipeline::premul); } if ( needsClamp0) { p->append(SkRasterPipeline::clamp_0); } if ( needsClamp1) { p->append(SkRasterPipeline::clamp_a); } - return true; } sk_sp<SkColorFilter> |