commit 7d8b397a4dc9f1350b6fdd28db74c64a7f00fb4b
author Brian Salomon <bsalomon@google.com> Tue Nov 26 22:34:44 2019 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Nov 27 18:55:23 2019 +0000
tree affb82caebcbf28ed5ede20d9a4c728be73b891f
parent 64331db0a412878caee7380152d2c7f30623fbe0

Changes to GrCoordTransform and local coords

*Don't track vertex-shader status per-coord transform, just at FP level

*Remove outdated "in processor" checks

*Adjust "uses local coords" on FP to consider whether coord transforms
 actually transform local coords or custom coords

*Remove unused accessMatrix() method

*Don't track normalization and y-flipping separately from proxy

*Rename some things for clarity

*Simplify FPCoordTransformHandler

Change-Id: Ic493afc82bd949bbab177d33111a1942e33f88a9
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/256101
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>

src/sksl/SkSLHCodeGenerator.cpp

diff --git a/src/sksl/SkSLHCodeGenerator.cpp b/src/sksl/SkSLHCodeGenerator.cpp
index 6c41e74..0790a1e 100644
--- a/src/sksl/SkSLHCodeGenerator.cpp
+++ b/src/sksl/SkSLHCodeGenerator.cpp
@@ -282,7 +282,7 @@
             this->writef("            %s_index = this->numChildProcessors();",
                          FieldName(String(param->fName).c_str()).c_str());
             if (fSectionAndParameterHelper.hasCoordOverrides(*param)) {
-                this->writef("            %s->setComputeLocalCoordsInVertexShader(false);",
+                this->writef("            %s->setSampledWithExplicitCoords(true);",
                              String(param->fName).c_str());
             }
             this->writef("            this->registerChildProcessor(std::move(%s));",