diff options
| author |  Mike Klein <mtklein@chromium.org> | 2017-06-09 15:58:37 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-06-09 21:07:40 +0000 |
| commit | cb3ceb79cb0dba69ee3a10c42b268b85ca71489b (patch) | |
| tree | 4b26bc14ca24031a7fcad7e8c713847783ad3ee5 /src/shaders/SkComposeShader.cpp | |
| parent | 06e547cb5f43dcc829f7f6a2ca773cfd602b32da (diff) | |
CanOverflow -> ClampIfNeeded
Centralize the clamping in ClampIfNeeded, and switch it to clamp_1.
Change-Id: I056ac029d3ddc81e3f0f194f2b9c9924cad4caae
Reviewed-on: https://skia-review.googlesource.com/19401
Commit-Queue: Mike Klein <mtklein@google.com>
Reviewed-by: Mike Reed <reed@google.com>
Diffstat (limited to 'src/shaders/SkComposeShader.cpp')
| -rw-r--r-- | src/shaders/SkComposeShader.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/shaders/SkComposeShader.cpp b/src/shaders/SkComposeShader.cpp index 1eae163c97..7b3714003c 100644 --- a/src/shaders/SkComposeShader.cpp +++ b/src/shaders/SkComposeShader.cpp @@ -110,12 +110,7 @@ bool SkComposeShader::onAppendStages(SkRasterPipeline* pipeline, SkColorSpace* d pipeline->append(SkRasterPipeline::load_rgba, storage->fRGBA); if (!this->isJustLerp()) { - // Idea: should time this, and see if it helps to have custom versions of the overflow modes - // that do their own clamping, avoiding the overhead of an extra stage. SkBlendMode_AppendStages(fMode, pipeline); - if (SkBlendMode_CanOverflow(fMode)) { - pipeline->append(SkRasterPipeline::clamp_a); - } } if (!this->isJustMode()) { pipeline->append(SkRasterPipeline::lerp_1_float, &fLerpT); |