add SkMatrixPriv for specialized helpers

add SkMatrixPriv for specialized helpers

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

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

[reed1, 2016-08-21 21:04:44]

Description was changed from

==========
add SkMatrixPriv for specialized helpers

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

to

==========
add SkMatrixPriv for specialized helpers

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

[reed1, 2016-08-21 21:04:59]

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

[commit-bot: I haz the power, 2016-08-21 21:05:03]

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

[reed1, 2016-08-21 21:15:08]

reed@google.com changed reviewers:
+ bsalomon@google.com

[reed1, 2016-08-21 21:15:08]

I changed the gm simplerects to have a translate (instead of identity ctm) and
it showed several matrix hotspots -- which I think I've caught.

[commit-bot: I haz the power, 2016-08-21 21:26:52]

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

[commit-bot: I haz the power, 2016-08-21 21:26:53]

Dry run: This issue passed the CQ dry run.

[reed1, 2016-08-21 21:37:45]

There are a few gm diffs (mostly in the low bit or 2), esp. in the bleed gms
(only on gpu backed naturally). Do we think this is understandable given my
change?

[bsalomon, 2016-08-22 13:29:41]

Slight GM changes are not surprising. I like this idea. lgtm.

[reed1, 2016-08-22 13:38:39]

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

[reed1, 2016-08-22 13:38:39]

Guess we'll cross our fingers on the DEPS roll ...

[reed1, 2016-08-22 13:38:47]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2016-08-22 13:38:55]

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-22 13:39:51]

Description was changed from

==========
add SkMatrixPriv for specialized helpers

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

to

==========
add SkMatrixPriv for specialized helpers

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

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

[commit-bot: I haz the power, 2016-08-22 13:39:52]

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

[mtklein, 2016-08-22 13:40:27]

On 2016/08/22 13:38:39, reed1 wrote:
> Guess we'll cross our fingers on the DEPS roll ...

(You can trigger Blink layout test trybots here now...)

[bungeman-skia, 2016-08-22 13:54:35]

Description was changed from

==========
add SkMatrixPriv for specialized helpers

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

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

to

==========
add SkMatrixPriv for specialized helpers

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

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

[bungeman-skia, 2016-08-22 14:25:34]

On 2016/08/22 13:40:27, mtklein wrote:
> On 2016/08/22 13:38:39, reed1 wrote:
> > Guess we'll cross our fingers on the DEPS roll ...
> 
> (You can trigger Blink layout test trybots here now...)

And... it probably won't matter, since 'analyze' apparently doesn't think a
change to SkMatrix could possibly affect layout tests.