diff options
| author |  Mike Klein <mtklein@chromium.org> | 2018-01-26 15:05:00 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-01-26 21:13:10 +0000 |
| commit | 05d7416b371fe1db9faef5a3623e54e109977c2c (patch) | |
| tree | ac3e52c3a6173a7a6c42e00b46d2a63c52d74b35 /src/core/SkColorSpace_XYZ.cpp | |
| parent | 71eabfe9e99e294e69b9b396ca6c901ac56fec38 (diff) | |
make SkGammas less bad
Data's existing operator== is terribly broken.
Instead replace it with what everyone really
wants to know, allChannelsSame().
I'm not quite ready to get rid of SkGammas yet,
but we're getting close.
Bug: chromium:799834
Change-Id: Ibcfd252e380b15fba53eb707f58304a50e90ce9f
Reviewed-on: https://skia-review.googlesource.com/100320
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src/core/SkColorSpace_XYZ.cpp')
| -rw-r--r-- | src/core/SkColorSpace_XYZ.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/core/SkColorSpace_XYZ.cpp b/src/core/SkColorSpace_XYZ.cpp index 105ed0f425..ed1446f8e9 100644 --- a/src/core/SkColorSpace_XYZ.cpp +++ b/src/core/SkColorSpace_XYZ.cpp @@ -65,7 +65,7 @@ bool SkColorSpace_XYZ::onIsNumericalTransferFn(SkColorSpaceTransferFn* coeffs) c } SkASSERT(fGammas); - if (fGammas->data(0) != fGammas->data(1) || fGammas->data(0) != fGammas->data(2)) { + if (!fGammas->allChannelsSame()) { return false; } |