diff options
| author |  Matt Sarett <msarett@google.com> | 2017-03-29 20:00:18 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-03-30 00:41:48 +0000 |
| commit | eeb3cb1286c5ca8104da62fb36b07bb0fe04aee5 (patch) | |
| tree | c3f9dc0ba32f113715f7ce24d6fcdcdc2f164c2d /src/core/SkColorSpaceXform.cpp | |
| parent | 7c4af54b7c46dd63f14f40bd15e3c2da9c0c86e2 (diff) | |
Reverse clamp logic in SkColorSpaceXform
Follow-up from https://skia-review.googlesource.com/c/10581/
Bug: skia:
Change-Id: I499f0908e1375854202c1cfd476827500fc89237
Reviewed-on: https://skia-review.googlesource.com/10645
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src/core/SkColorSpaceXform.cpp')
| -rw-r--r-- | src/core/SkColorSpaceXform.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/core/SkColorSpaceXform.cpp b/src/core/SkColorSpaceXform.cpp index 82b442b8f6..52a685194c 100644 --- a/src/core/SkColorSpaceXform.cpp +++ b/src/core/SkColorSpaceXform.cpp @@ -1227,9 +1227,8 @@ bool SkColorSpaceXform_XYZ<kCSM> if (kNone_ColorSpaceMatch == kCSM) { pipeline.append(SkRasterPipeline::matrix_3x4, fSrcToDst); - if (kRGBA_8888_ColorFormat == dstColorFormat || - kBGRA_8888_ColorFormat == dstColorFormat || - kBGR_565_ColorFormat == dstColorFormat) + if (kRGBA_F16_ColorFormat != dstColorFormat && + kRGBA_F32_ColorFormat != dstColorFormat) { bool need_clamp_0, need_clamp_1; analyze_3x4_matrix(fSrcToDst, &need_clamp_0, &need_clamp_1); |