commit 7633477b649c36dd2a03c21d799f66193a341182
author Jim Van Verth <jvanverth@google.com> Fri May 05 16:46:05 2017 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri May 05 21:13:31 2017 +0000
tree 443e97a51271f40210b0888de26c61ca0c60ac14
parent 6d342285a4546b54cb17570aae7eeb8a123c81ae

Fix color blending for analytic shadows

Change-Id: I74e8c385a87a3f397b2df53679fcfd9120d82954
Reviewed-on: https://skia-review.googlesource.com/15645
Commit-Queue: Jim Van Verth <jvanverth@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>

src/effects/shadows/SkAmbientShadowMaskFilter.cpp

diff --git a/src/effects/shadows/SkAmbientShadowMaskFilter.cpp b/src/effects/shadows/SkAmbientShadowMaskFilter.cpp
index 7578ae3..3416835 100644
--- a/src/effects/shadows/SkAmbientShadowMaskFilter.cpp
+++ b/src/effects/shadows/SkAmbientShadowMaskFilter.cpp
@@ -215,8 +215,7 @@
 
     GrPaint newPaint(paint);
     GrColor4f color = newPaint.getColor4f();
-    color.fRGBA[3] *= fAmbientAlpha;
-    newPaint.setColor4f(color);
+    newPaint.setColor4f(color.mulByScalar(fAmbientAlpha));
     if (SkToBool(fFlags & SkShadowFlags::kTransparentOccluder_ShadowFlag)) {
         // set a large inset to force a fill
         devSpaceInsetWidth = ambientRRect.width();

src/effects/shadows/SkSpotShadowMaskFilter.cpp

diff --git a/src/effects/shadows/SkSpotShadowMaskFilter.cpp b/src/effects/shadows/SkSpotShadowMaskFilter.cpp
index 0716d9b..a51aa56 100644
--- a/src/effects/shadows/SkSpotShadowMaskFilter.cpp
+++ b/src/effects/shadows/SkSpotShadowMaskFilter.cpp
@@ -297,8 +297,7 @@
     }
 
     GrColor4f color = paint.getColor4f();
-    color.fRGBA[3] *= fSpotAlpha;
-    paint.setColor4f(color);
+    paint.setColor4f(color.mulByScalar(fSpotAlpha));
     rtContext->drawShadowRRect(clip, std::move(paint), viewMatrix, spotShadowRRect,
                                devSpaceSpotBlur, insetWidth);
 

src/gpu/gl/GrGLGpu.cpp

diff --git a/src/gpu/gl/GrGLGpu.cpp b/src/gpu/gl/GrGLGpu.cpp
index 8d76fae..3f200bb 100644
--- a/src/gpu/gl/GrGLGpu.cpp
+++ b/src/gpu/gl/GrGLGpu.cpp
@@ -191,6 +191,9 @@
     if (!glInterface) {
         return nullptr;
     }
+#ifdef USE_NSIGHT
+    const_cast<GrContextOptions&>(options).fSuppressPathRendering = true;
+#endif
     GrGLContext* glContext = GrGLContext::Create(glInterface.get(), options);
     if (glContext) {
         return new GrGLGpu(glContext, context);