diff options
| author |  Mike Klein <mtklein@chromium.org> | 2018-05-24 13:59:58 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-05-24 18:36:41 +0000 |
| commit | 7d8276cbabab1357d0a5ac4f6bc37e4f6b0a80d3 (patch) | |
| tree | 381a6f3db9f7cf83738ca0c80b8de491696a1885 /tests | |
| parent | 6968f9ce54a627254ff91206df6c78de99930c89 (diff) | |
skip gamut transform when identity
Change-Id: I3bb3ec19ee856ce4070a58a57b3bb8e8a170a5b9
Reviewed-on: https://skia-review.googlesource.com/130024
Commit-Queue: Mike Klein <mtklein@chromium.org>
Auto-Submit: Mike Klein <mtklein@chromium.org>
Reviewed-by: Brian Osman <brianosman@google.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/SkColorSpaceXformStepsTest.cpp | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/tests/SkColorSpaceXformStepsTest.cpp b/tests/SkColorSpaceXformStepsTest.cpp index 47e5e1b90b..900eb4a9cd 100644 --- a/tests/SkColorSpaceXformStepsTest.cpp +++ b/tests/SkColorSpaceXformStepsTest.cpp @@ -110,25 +110,25 @@ DEF_TEST(SkColorSpaceXformSteps, r) { // These eight cases transform between color spaces with different // transfer functions and the same gamut. Optimization here is limited // to skipping the gamut_transform step: | - // v This column of true can all become false. - { srgb_N, srgb22_N, true,true,false, true,true,true, false,false }, - { srgb22_N, srgb_N, true,true,false, true,true,true, false,false }, + // v This column has all become false. + { srgb_N, srgb22_N, true,true,false, false,true,true, false,false }, + { srgb22_N, srgb_N, true,true,false, false,true,true, false,false }, - { srgb_L, srgb22_L, false,true,true, true,false,true, true,true }, - { srgb22_L, srgb_L, false,true,true, true,false,true, true,true }, + { srgb_L, srgb22_L, false,true,true, false,false,true, true,true }, + { srgb22_L, srgb_L, false,true,true, false,false,true, true,true }, - { srgb_N, srgb22_L, true,true,false, true,false,true, true,true }, - { srgb22_N, srgb_L, true,true,false, true,false,true, true,true }, + { srgb_N, srgb22_L, true,true,false, false,false,true, true,true }, + { srgb22_N, srgb_L, true,true,false, false,false,true, true,true }, - { srgb_L, srgb22_N, false,true,true, true,true,true, false,false }, - { srgb22_L, srgb_N, false,true,true, true,true,true, false,false }, + { srgb_L, srgb22_N, false,true,true, false,true,true, false,false }, + { srgb22_L, srgb_N, false,true,true, false,true,true, false,false }, // These four test cases test drawing in the same color space. - // There is lots of room for optimization here, but none implemented yet. - { srgb_N, srgb_N, true,true,false, true,true,true, false,false }, // a.k.a legacy 8888 - { srgb_L, srgb_L, false,true,true, true,false,true, true,true }, // <canvas> use case - { srgb_N, srgb_L, true,true,false, true,false,true, true,true }, - { srgb_L, srgb_N, false,true,true, true,true,true, false,false }, + // There is lots of room for optimization here. + { srgb_N, srgb_N, true,true,false, false,true,true, false,false }, // a.k.a legacy 8888 + { srgb_L, srgb_L, false,true,true, false,false,true, true,true }, // <canvas> use case + { srgb_N, srgb_L, true,true,false, false,false,true, true,true }, + { srgb_L, srgb_N, false,true,true, false,true,true, false,false }, // TODO: versions of above crossing in linear transfer functions }; |