commit 1298bc46ac779263485dd56b6c9efccb84150ef1
author Brian Osman <brianosman@google.com> Tue Jun 30 13:39:35 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Jul 01 16:37:43 2020 +0000
tree 73654662fde350b6aeb66671e50bf8c3b627b67e
parent f8f45d91b2e7e841719ed1d80ee74f68aeb8d0ec

Change SampleMatrix to SampleUsage

It now tracks all sample calls of a child (matrix, explicit coords,
pass through). There is now just one registerChild() call, and the
sampling pattern of that child is fully determined by the SampleUsage
parameter.

Change-Id: Iaadcd325fca64a59f24192aadd06923c66362181
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/299875
Reviewed-by: Michael Ludwig <michaelludwig@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>

src/gpu/effects/GrMatrixConvolutionEffect.cpp

diff --git a/src/gpu/effects/GrMatrixConvolutionEffect.cpp b/src/gpu/effects/GrMatrixConvolutionEffect.cpp
index 12443af..6fb6b42 100644
--- a/src/gpu/effects/GrMatrixConvolutionEffect.cpp
+++ b/src/gpu/effects/GrMatrixConvolutionEffect.cpp
@@ -292,9 +292,9 @@
         , fGain(SkScalarToFloat(gain))
         , fBias(SkScalarToFloat(bias) / 255.0f)
         , fConvolveAlpha(convolveAlpha) {
-    this->registerExplicitlySampledChild(std::move(child));
+    this->registerChild(std::move(child), SkSL::SampleUsage::Explicit());
     if (kernelFP) {
-        this->registerExplicitlySampledChild(std::move(kernelFP));
+        this->registerChild(std::move(kernelFP), SkSL::SampleUsage::Explicit());
     }
     fKernelOffset = {static_cast<float>(kernelOffset.x()),
                      static_cast<float>(kernelOffset.y())};