commit 8e17556349132a46c02fbabcaf74b63f521528e0
author benjaminwagner <benjaminwagner@google.com> Tue Feb 16 10:09:40 2016 -0800
committer Commit bot <commit-bot@chromium.org> Tue Feb 16 10:09:40 2016 -0800
tree bafcbb7188ab624191337a4f636eb55f1d80aa68
parent e71dc3fd0d7e8a9ff8ffbb56c5c2a9daeed9df49

Change SkMatrix::fixedStepInX to return SkVector (of SkScalar) rather than SkFixed.

All users were immediately converting to SkScalar or SkFixed3232.

This method is not used in Chromium, Android, or Google3.

BUG=skia:4632
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1692013002

Review URL: https://codereview.chromium.org/1692013002

src/effects/gradients/SkRadialGradient.cpp

diff --git a/src/effects/gradients/SkRadialGradient.cpp b/src/effects/gradients/SkRadialGradient.cpp
index 7e7ac7a..020c25e 100644
--- a/src/effects/gradients/SkRadialGradient.cpp
+++ b/src/effects/gradients/SkRadialGradient.cpp
@@ -206,11 +206,9 @@
         SkScalar sdy = fDstToIndex.getSkewY();
 
         if (fDstToIndexClass == kFixedStepInX_MatrixClass) {
-            SkFixed storage[2];
-            (void)fDstToIndex.fixedStepInX(SkIntToScalar(y),
-                                           &storage[0], &storage[1]);
-            sdx = SkFixedToScalar(storage[0]);
-            sdy = SkFixedToScalar(storage[1]);
+            const auto step = fDstToIndex.fixedStepInX(SkIntToScalar(y));
+            sdx = step.fX;
+            sdy = step.fY;
         } else {
             SkASSERT(fDstToIndexClass == kLinear_MatrixClass);
         }