aboutsummaryrefslogtreecommitdiffhomepage
path: root/src
diff options
context:
space:
mode:
authorGravatar Stan Iliev <stani@google.com>2017-07-26 15:07:05 -0400
committerGravatar Skia Commit-Bot <skia-commit-bot@chromium.org>2017-07-26 19:31:33 +0000
commitf695b63841fb4c55353bcf98932985e0deea3112 (patch)
treebdc4f93a760f3128531be36a81f252efafb19893 /src
parent789966d8ddb6f6dba7f889202cee621c76a1057e (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.cpp3
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