diff options
author | 2016-10-27 11:29:36 -0400 | |
---|---|---|
committer | 2016-10-27 20:00:21 +0000 | |
commit | 130863ef51a2a94e5bdf87f344c0e892b2403985 (patch) | |
tree | 4c25fab02c00819fbda547e711630ba3fdd6abc8 /src/gpu/GrStyle.cpp | |
parent | 6fe16d3f58937957351fb347a7b1dffa0e0768bc (diff) |
Only clamp when we think our math requires it.
If we require our inputs are sound, in-gamut, premul colors (a in [0,1], r,g,b in [0,a]) then we should only need to clamp when the math we perform requires it. The safety clamps before each store are paranoia.
The main thing this pipeline handles right now that needs clamping is the plus transfermode.
This is either used to blend, where the clamp must come after the coverage lerp, or used via a mode color filter, where we have no choice but to clamp right at the end of the color filer.
This changes how the mode color filter draws with the plus transfermode. It didn't used to clamp at all. I think this is a bug fix.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4034
Change-Id: I3cbaade2127cc88c8782596f45749c4fe4b0e953
Reviewed-on: https://skia-review.googlesource.com/4034
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src/gpu/GrStyle.cpp')
0 files changed, 0 insertions, 0 deletions