Split distance vector into direction and magnitude components

Split distance vector into direction and magnitude components

The vector was split because the vector's users need it split, and we were creating it from the split components in the first place, so it made sense to skip that step.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2223053002

Committed: https://skia.googlesource.com/skia/+/84bca78ab462314ec48614ff50315f28d5467149

[dvonbeck, 2016-08-08 15:13:31]

Description was changed from

==========
Split distance vector into direction and magnitude components

The vector was split because the vector's users need it split, and we were
creating it from the split components in the first place, so it made sense to
skip that step.

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

to

==========
Split distance vector into direction and magnitude components

The vector was split because the vector's users need it split, and we were
creating it from the split components in the first place, so it made sense to
skip that step.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2223053002
==========

[dvonbeck, 2016-08-08 15:17:25]

dvonbeck@google.com changed reviewers:
+ egdaniel@google.com, robertphillips@google.com

[dvonbeck, 2016-08-08 15:17:26]

I split the vector generation at the GeoProc. Please let me know if you think I
should have separate variables for each component as opposed to packing them in
a vec3.

[egdaniel, 2016-08-08 17:53:04]

lgtm

[dvonbeck, 2016-08-08 17:54:40]

The CQ bit was checked by dvonbeck@google.com

[commit-bot: I haz the power, 2016-08-08 17:54:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-08 18:47:17]

Description was changed from

==========
Split distance vector into direction and magnitude components

The vector was split because the vector's users need it split, and we were
creating it from the split components in the first place, so it made sense to
skip that step.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2223053002
==========

to

==========
Split distance vector into direction and magnitude components

The vector was split because the vector's users need it split, and we were
creating it from the split components in the first place, so it made sense to
skip that step.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2223053002

Committed:
https://skia.googlesource.com/skia/+/84bca78ab462314ec48614ff50315f28d5467149
==========

[commit-bot: I haz the power, 2016-08-08 18:47:20]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/84bca78ab462314ec48614ff50315f28d5467149