Change SkMatrix::fixedStepInX to return SkVector rather than SkFixed.

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

Committed: https://skia.googlesource.com/skia/+/8e17556349132a46c02fbabcaf74b63f521528e0

[dogben, 2016-02-11 21:11:17]

Description was changed from

==========
Change SkMatrix::fixedStepInX to return SkScalar rather than SkFixed.

All users were immediately converting to SkScalar or SkFixed3232.

BUG=skia:4632
==========

to

==========
Change SkMatrix::fixedStepInX to return SkScalar rather than SkFixed.

All users were immediately converting to SkScalar or SkFixed3232.

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

[dogben, 2016-02-11 21:11:58]

The CQ bit was checked by benjaminwagner@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-02-11 21:12:10]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1692013002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1692013002/1

[dogben, 2016-02-11 21:13:55]

Description was changed from

==========
Change SkMatrix::fixedStepInX to return SkScalar rather than SkFixed.

All users were immediately converting to SkScalar or SkFixed3232.

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

to

==========
Change SkMatrix::fixedStepInX to return 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&is...
==========

[commit-bot: I haz the power, 2016-02-11 21:40:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-11 21:40:41]

Dry run: This issue passed the CQ dry run.

[dogben, 2016-02-11 21:43:35]

benjaminwagner@google.com changed reviewers:
+ mtklein@google.com, reed@google.com

[dogben, 2016-02-11 21:43:36]

There are GM diffs, so if you think I should commit this, I'll need some
hand-holding.

[reed1, 2016-02-11 21:48:00]

I like this trend!

Can we change the check into an assert inside the function, and then change the
signature to just return SkPoint ?

[reed1, 2016-02-11 21:48:50]

reed@google.com changed reviewers:
+ fmalita@chromium.org

[dogben, 2016-02-12 15:43:53]

The CQ bit was checked by benjaminwagner@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-02-12 15:44:05]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1692013002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1692013002/20001

[dogben, 2016-02-12 15:56:32]

Description was changed from

==========
Change SkMatrix::fixedStepInX to return 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&is...
==========

to

==========
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&is...
==========

[commit-bot: I haz the power, 2016-02-12 16:08:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-12 16:08:34]

Dry run: This issue passed the CQ dry run.

[dogben, 2016-02-12 16:09:49]

What do you think of this? I needed to add isFixedStepInX.

If you don't like it, I can revert.

[reed1, 2016-02-15 21:02:57]

lgtm

[dogben, 2016-02-16 17:38:09]

The CQ bit was checked by benjaminwagner@google.com

[commit-bot: I haz the power, 2016-02-16 17:38:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1692013002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1692013002/20001

[commit-bot: I haz the power, 2016-02-16 18:09:42]

Description was changed from

==========
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&is...
==========

to

==========
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&is...

Committed:
https://skia.googlesource.com/skia/+/8e17556349132a46c02fbabcaf74b63f521528e0
==========

[commit-bot: I haz the power, 2016-02-16 18:09:43]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/8e17556349132a46c02fbabcaf74b63f521528e0