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>

diff --git a/src/core/SkColorSpaceXform.cpp b/src/core/SkColorSpaceXform.cpp
index 82b442b..52a6851 100644 (file)
--- 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);