commit 2fade72a4ea2a6ca635137e7718b7d5ce6eab474
author Brian Salomon <bsalomon@google.com> Fri Mar 20 12:35:32 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Mar 20 17:44:45 2020 +0000
tree 0fda58d952fd124782ef6552cc04f87ba2bad7d4
parent 7b7b78cb9f1bd8b57fb849e5cae7f477e84d3cf6

Set correct coord transform visibility initially rather than update.

Aside from being simpler it also makes the matrix uniforms for
explicitly sampled FPs appear only in the FS and not both VS and FS.

Change-Id: I0f7a4e83b77decacfd0567600dd2d3bc8ce5240f
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/278137
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>

src/gpu/glsl/GrGLSLGeometryProcessor.cpp

diff --git a/src/gpu/glsl/GrGLSLGeometryProcessor.cpp b/src/gpu/glsl/GrGLSLGeometryProcessor.cpp
index 0eba2cd..0c493d0 100644
--- a/src/gpu/glsl/GrGLSLGeometryProcessor.cpp
+++ b/src/gpu/glsl/GrGLSLGeometryProcessor.cpp
@@ -87,12 +87,12 @@
         if (!fp.isSampledWithExplicitCoords() || !coordTransform.isNoOp()) {
             SkString strUniName;
             strUniName.printf("CoordTransformMatrix_%d", i);
-            fInstalledTransforms.push_back().fHandle = uniformHandler
-                                                               ->addUniform(kVertex_GrShaderFlag,
-                                                                            kFloat3x3_GrSLType,
-                                                                            strUniName.c_str(),
-                                                                            &matrixName)
-                                                               .toIndex();
+            auto flag = fp.isSampledWithExplicitCoords() ? kFragment_GrShaderFlag
+                                                         : kVertex_GrShaderFlag;
+            fInstalledTransforms.push_back().fHandle =
+                    uniformHandler
+                            ->addUniform(flag, kFloat3x3_GrSLType, strUniName.c_str(), &matrixName)
+                            .toIndex();
         } else {
             // Install a coord transform that will be skipped.
             fInstalledTransforms.push_back();