SkMatrix44 clarifications and clean-ups

SkMatrix44 clarifications and clean-ups

Fixed row/col major bug and added comments to the
header.

Made SkMatrix::I() thread safe using constexpr.

Added a test set3x3RowMajorf().

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

Committed: https://skia.googlesource.com/skia/+/c1a3e24918f99fc0b975111afb39dca38c50eb5c

[msarett, 2016-06-23 18:11:48]

Description was changed from

==========
SkMatrix44 clarifications and clean-ups

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

to

==========
SkMatrix44 clarifications and clean-ups

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

[msarett, 2016-06-23 18:14:57]

Patchset #1 (id:1) has been deleted

[msarett, 2016-06-23 18:16:17]

Description was changed from

==========
SkMatrix44 clarifications and clean-ups

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

to

==========
SkMatrix44 clarifications and clean-ups

Fixed row/col major bug and added comments to the
header.

Made SkMatrix::I() thread safe using constexpr.

Added a test set3x3RowMajorf().

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

[msarett, 2016-06-23 18:17:49]

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

[msarett, 2016-06-23 18:17:50]

I didn't do anything too exciting here, mostly because I think SkMatrix44 makes
a good amount of sense as is.

[reed1, 2016-06-23 18:37:29]

lgtm

[reed1, 2016-06-23 18:37:29]

lgtm

[msarett, 2016-06-23 18:55:10]

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

[commit-bot: I haz the power, 2016-06-23 18:55:22]

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

[commit-bot: I haz the power, 2016-06-23 19:16:01]

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

[commit-bot: I haz the power, 2016-06-23 19:16:02]

Dry run: This issue passed the CQ dry run.

[msarett, 2016-06-23 19:41:21]

The CQ bit was checked by msarett@google.com

[commit-bot: I haz the power, 2016-06-23 19:41:32]

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

[commit-bot: I haz the power, 2016-06-23 19:42:31]

Description was changed from

==========
SkMatrix44 clarifications and clean-ups

Fixed row/col major bug and added comments to the
header.

Made SkMatrix::I() thread safe using constexpr.

Added a test set3x3RowMajorf().

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

to

==========
SkMatrix44 clarifications and clean-ups

Fixed row/col major bug and added comments to the
header.

Made SkMatrix::I() thread safe using constexpr.

Added a test set3x3RowMajorf().

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

Committed:
https://skia.googlesource.com/skia/+/c1a3e24918f99fc0b975111afb39dca38c50eb5c
==========

[commit-bot: I haz the power, 2016-06-23 19:42:33]

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