commit 23e7777f421d11992e8245771641141aa273556b
author Mike Klein <mtklein@google.com> Fri Jul 27 10:34:14 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Jul 27 14:42:26 2018 +0000
tree 9dca58c312a6d302be811d06fadd65a339414cc6
parent 4642d9fe69e1ace1bb565a39f38fea9f776d8adb

clamp before premul

When alpha is <1, it's possible to fit r,g,b >1 into premul unorms,
and that subverts the expectations users who consume the output of
skcms_Transform().

When working with unorm pixels, downstream consumers will assume the
r,g,b values are also within [0,1], i.e. are less than the alpha
channel.  Put another way, they assume the colors are in gamut.

So clamp earlier, before premul.  (The reordering has no effect on
force_opaque or swap_rb.)

Bug: chromium:867813
Bug: chromium:862908
Change-Id: I553b42df06482a4e3cd3a9377c2adddb8ff3d1bc
Reviewed-on: https://skia-review.googlesource.com/143710
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
Auto-Submit: Mike Klein <mtklein@google.com>