diff options
author | Mike Klein <mtklein@chromium.org> | 2018-07-11 14:49:51 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-07-11 20:14:38 +0000 |
commit | 4eebd9eed06039d265f06edb759765731f963271 (patch) | |
tree | a60d5bf110250ad75593977b39fdf93a019c1fff /src/core/SkColorSpaceXformSteps.cpp | |
parent | c0a27081722eaebd005cf9c2eb78d7d1e4db7e2c (diff) |
collapse parametric_{r,g,b} into parametric, remove _a and gamma_dst
parametric_a and gamma_dst were unused outside of unit tests.
In all other cases, we always use parametric_{r,g,b} together
and always pass them the same argument. So we can collapse
them into a single stage like gamma and to/from_srgb.
Change-Id: I08cea896c7744f97b4f4bf9e029f5d643e45e177
Reviewed-on: https://skia-review.googlesource.com/140576
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Brian Osman <brianosman@google.com>
Diffstat (limited to 'src/core/SkColorSpaceXformSteps.cpp')
-rw-r--r-- | src/core/SkColorSpaceXformSteps.cpp | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/core/SkColorSpaceXformSteps.cpp b/src/core/SkColorSpaceXformSteps.cpp index 77a9c6b05d..3abdb2f05a 100644 --- a/src/core/SkColorSpaceXformSteps.cpp +++ b/src/core/SkColorSpaceXformSteps.cpp @@ -115,9 +115,7 @@ void SkColorSpaceXformSteps::apply(SkRasterPipeline* p) const { srcTF.fF == 0) { p->append(SkRasterPipeline::gamma, &srcTF.fG); } else { - p->append(SkRasterPipeline::parametric_r, &srcTF); - p->append(SkRasterPipeline::parametric_g, &srcTF); - p->append(SkRasterPipeline::parametric_b, &srcTF); + p->append(SkRasterPipeline::parametric, &srcTF); } } if (flags.gamut_transform) { @@ -134,9 +132,7 @@ void SkColorSpaceXformSteps::apply(SkRasterPipeline* p) const { dstTFInv.fF == 0) { p->append(SkRasterPipeline::gamma, &dstTFInv.fG); } else { - p->append(SkRasterPipeline::parametric_r, &dstTFInv); - p->append(SkRasterPipeline::parametric_g, &dstTFInv); - p->append(SkRasterPipeline::parametric_b, &dstTFInv); + p->append(SkRasterPipeline::parametric, &dstTFInv); } } if (flags.premul) { p->append(SkRasterPipeline::premul); } |