From 8e17556349132a46c02fbabcaf74b63f521528e0 Mon Sep 17 00:00:00 2001 From: benjaminwagner Date: Tue, 16 Feb 2016 10:09:40 -0800 Subject: 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 --- include/core/SkMatrix.h | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'include') diff --git a/include/core/SkMatrix.h b/include/core/SkMatrix.h index 0ebe3280e2..adebada445 100644 --- a/include/core/SkMatrix.h +++ b/include/core/SkMatrix.h @@ -591,11 +591,16 @@ public: return GetMapPtsProc(this->getType()); } + /** Returns true if the matrix can be stepped in X (not complex + perspective). + */ + bool isFixedStepInX() const; + /** If the matrix can be stepped in X (not complex perspective) - then return true and if step[XY] is not null, return the step[XY] value. - If it cannot, return false and ignore step. + then return the step value. + If it cannot, behavior is undefined. */ - bool fixedStepInX(SkScalar y, SkFixed* stepX, SkFixed* stepY) const; + SkVector fixedStepInX(SkScalar y) const; /** Efficient comparison of two matrices. It distinguishes between zero and * negative zero. It will return false when the sign of zero values is the -- cgit v1.2.3