diff options
author | Stan Iliev <stani@google.com> | 2017-07-26 15:07:05 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-07-26 19:31:33 +0000 |
commit | f695b63841fb4c55353bcf98932985e0deea3112 (patch) | |
tree | bdc4f93a760f3128531be36a81f252efafb19893 /src | |
parent | 789966d8ddb6f6dba7f889202cee621c76a1057e (diff) |
Don't clamp in GrGLNonlinearColorSpaceXformEffect to support scRGB
Remove color clamp to allow for negative and greater than 1.0
values, when converting to scRGB.
Bug: b/62347704
Change-Id: I5b811d5dd464cf3bb126ad435a7d27c4563fdacc
Reviewed-on: https://skia-review.googlesource.com/26862
Commit-Queue: Stan Iliev <stani@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/gpu/effects/GrNonlinearColorSpaceXformEffect.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/gpu/effects/GrNonlinearColorSpaceXformEffect.cpp b/src/gpu/effects/GrNonlinearColorSpaceXformEffect.cpp index 8153725460..d26377b691 100644 --- a/src/gpu/effects/GrNonlinearColorSpaceXformEffect.cpp +++ b/src/gpu/effects/GrNonlinearColorSpaceXformEffect.cpp @@ -88,9 +88,8 @@ public: // 3: Apply gamut matrix if (gamutXformName) { - // Color is unpremultiplied at this point, so clamp to [0, 1] fragBuilder->codeAppendf( - "color.rgb = clamp((%s * vec4(color.rgb, 1.0)).rgb, 0.0, 1.0);", gamutXformName); + "color.rgb = (%s * vec4(color.rgb, 1.0)).rgb;", gamutXformName); } // 4: Apply dst transfer fn |