commit 66b500a07cddd85d8fe5441d73cf1f161b0ebfce
author Brian Salomon <bsalomon@google.com> Mon Aug 02 12:37:14 2021 -0400
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> Mon Aug 02 17:41:45 2021 +0000
tree 441b0105766583e3dda8adefdc8f5a2d3f677bf1
parent d7437eec2e9718eda8c8f5cb94ddcef1005c0bfb

Remove explicit sample flag from GrFP.

Instead expand the map that is computed by GrGLSLGP to include a bool
that indicates whether each FP requires coords or not.

Now GrGLSLGP is solely responsible for determining which FPs take coords
and inserting any varyings. The rest of the system follows its lead when
generating FP functions and call sites.

Bug: skia:12198

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

tests/SkRuntimeEffectTest.cpp

diff --git a/tests/SkRuntimeEffectTest.cpp b/tests/SkRuntimeEffectTest.cpp
index c139588..d5409e4 100644
--- a/tests/SkRuntimeEffectTest.cpp
+++ b/tests/SkRuntimeEffectTest.cpp
@@ -893,8 +893,8 @@
                                  "child", std::move(child));
         REPORTER_ASSERT(r, fp);
 
-        REPORTER_ASSERT(r, fp->childProcessor(0)->isSampledWithExplicitCoords() == expectExplicit);
-        REPORTER_ASSERT(r, fp->referencesSampleCoords() == expectReferencesSampleCoords);
+        REPORTER_ASSERT(r, fp->childProcessor(0)->sampleUsage().isExplicit() == expectExplicit);
+        REPORTER_ASSERT(r, fp->usesSampleCoords() == expectReferencesSampleCoords);
     };
 
     // Cases where our optimization is valid, and works: