commit 41f9c3cde9eac4a9d142b2c1fc86124988038262
author Brian Salomon <bsalomon@google.com> Mon Mar 25 11:06:12 2019 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Mon Mar 25 16:51:58 2019 +0000
tree c8f183229e1138e6cf9814dc058dfe0c863fe443
parent d1630203fb6a10808d255b02e5bd29ffd444cecc

Avoid special shader for opaque alpha when drawing simple rectangles.

It's a wasted shader compiler and also seems to cause a rendering bug
in Windows Chrome on ANGLE.

Bug: chromium:942565
Change-Id: I56373ac596d4d1548d6c86cda7127bf12190a0ae
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/203383
Reviewed-by: Michael Ludwig <michaelludwig@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>

src/gpu/GrProcessorSet.cpp

diff --git a/src/gpu/GrProcessorSet.cpp b/src/gpu/GrProcessorSet.cpp
index 099c05e..94dd240 100644
--- a/src/gpu/GrProcessorSet.cpp
+++ b/src/gpu/GrProcessorSet.cpp
@@ -235,6 +235,7 @@
     }
     fFragmentProcessorOffset = colorFPsToEliminate;
     fColorFragmentProcessorCnt -= colorFPsToEliminate;
+    analysis.fHasColorFragmentProcessor = (fColorFragmentProcessorCnt != 0);
 
     bool hasMixedSampledCoverage = (GrFSAAType::kMixedSamples == fsaaType)
             && !userStencil->testAlwaysPasses((clip) ? clip->hasStencilClip() : false);