commit b10a6625ba59fce8cd673e0d9bc3c15e3a4196f6
author Brian Salomon <bsalomon@google.com> Thu Feb 20 13:36:01 2020 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Feb 20 19:34:46 2020 +0000
tree f2c6bdd5985c17afe59d4de075a4a455ffa57872
parent 62076977a0b7f111edd1d07e8af3e2b8e6eed199

GrFP::coordTransformsApplyToLocalCoords() -> GrFP::isSampledWithExplicitCoords()

Also rename and flip meaning of the internal flag.

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

src/gpu/glsl/GrGLSLFragmentProcessor.cpp

diff --git a/src/gpu/glsl/GrGLSLFragmentProcessor.cpp b/src/gpu/glsl/GrGLSLFragmentProcessor.cpp
index bc7dd60..8d08de5 100644
--- a/src/gpu/glsl/GrGLSLFragmentProcessor.cpp
+++ b/src/gpu/glsl/GrGLSLFragmentProcessor.cpp
@@ -30,7 +30,7 @@
     // does want to sample with explicit coords, that property is propagated (recursively) to all
     // children, and we need to supply explicit coords. So we propagate our own "_coords" (this is
     // the name of our explicit coords parameter generated in the helper function).
-    if (!args.fFp.coordTransformsApplyToLocalCoords() && skslCoords.length() == 0) {
+    if (args.fFp.isSampledWithExplicitCoords() && skslCoords.length() == 0) {
         skslCoords = "_coords";
     }
 
@@ -59,7 +59,7 @@
 
     // If the fragment processor is invoked with overridden coordinates, it must *always* be invoked
     // with overridden coords.
-    SkASSERT(childProc.coordTransformsApplyToLocalCoords() == (skslCoords.length() == 0));
+    SkASSERT(childProc.isSampledWithExplicitCoords() == !skslCoords.empty());
 
     // Produce a string containing the call to the helper function
     SkString result = SkStringPrintf("%s(%s", fFunctionNames[childIndex].c_str(),