commit 50b2084bd15e7aea416dc8c6fd2373864c50f023
author Florin Malita <fmalita@chromium.org> Sat Jul 29 19:08:28 2017 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Sun Jul 30 14:24:21 2017 +0000
tree a68b832e4eb3e9e8cd8c463bccab8b9836cd6fff
parent c25437da5c6a3e5766c7811cfd484c5ef1b2861e

Gradient stages cleanup

Hoist the matrix setup into SkGradientShaderBase.

Drop the bool return for adjustMatrixAndAppendStages(), and rename to
appendGradientStages().

Change-Id: Iad412f638f86223e5af46a7d3dea6f768f52bcad
Reviewed-on: https://skia-review.googlesource.com/26427
Commit-Queue: Florin Malita <fmalita@chromium.org>
Reviewed-by: Mike Reed <reed@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>

src/shaders/gradients/SkLinearGradient.cpp

diff --git a/src/shaders/gradients/SkLinearGradient.cpp b/src/shaders/gradients/SkLinearGradient.cpp
index 605ae54..32b2ba5 100644
--- a/src/shaders/gradients/SkLinearGradient.cpp
+++ b/src/shaders/gradients/SkLinearGradient.cpp
@@ -79,12 +79,9 @@
                            : nullptr;
 }
 
-bool SkLinearGradient::adjustMatrixAndAppendStages(SkArenaAlloc*,
-                                                   SkMatrix* matrix,
-                                                   SkRasterPipeline*,
-                                                   SkRasterPipeline*) const {
-    matrix->postConcat(fPtsToUnit);
-    return true;
+void SkLinearGradient::appendGradientStages(SkArenaAlloc*, SkRasterPipeline*,
+                                            SkRasterPipeline*) const {
+    // No extra stage needed for linear gradients.
 }
 
 sk_sp<SkShader> SkLinearGradient::onMakeColorSpace(SkColorSpaceXformer* xformer) const {