drawBitmapRect() should not touch the CTM when mask filters are present

drawBitmapRect() should not touch the CTM when mask filters are present

Blur sigma calculations are CTM dependent, so we cannot take the
drawBitmap() fast path in the presence of mask filters.

BUG=skia:5682
R=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2286873002

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

[f(malita), 2016-08-26 19:11:20]

Description was changed from

==========
drawBitmapRect() should not touch the CTM when mask filters are present

Blur sigma calculations are CTM dependent, so we cannot take the
drawBitmap() fast path in the presence of mask filters.

BUG=skia:5682
R=reed@google.com
==========

to

==========
drawBitmapRect() should not touch the CTM when mask filters are present

Blur sigma calculations are CTM dependent, so we cannot take the
drawBitmap() fast path in the presence of mask filters.

BUG=skia:5682
R=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2286873002
==========

[f(malita), 2016-08-26 19:18:17]

Also tweak an existing GM to cover subsetting.

[f(malita), 2016-08-26 19:18:22]

The CQ bit was checked by fmalita@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-26 19:18:30]

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

[f(malita), 2016-08-26 19:19:11]

https://codereview.chromium.org/2286873002/diff/20001/src/core/SkBitmapDevice...
File src/core/SkBitmapDevice.cpp (right):

https://codereview.chromium.org/2286873002/diff/20001/src/core/SkBitmapDevice...
src/core/SkBitmapDevice.cpp:237: if (!paint.getMaskFilter()) {
Is there anything else downstream that may care about the CTM?

[commit-bot: I haz the power, 2016-08-26 19:36:54]

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

[commit-bot: I haz the power, 2016-08-26 19:36:55]

Dry run: This issue passed the CQ dry run.

[reed1, 2016-08-26 19:41:27]

lgtm

[f(malita), 2016-08-26 20:03:06]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2016-08-26 20:03:13]

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-26 20:04:16]

Description was changed from

==========
drawBitmapRect() should not touch the CTM when mask filters are present

Blur sigma calculations are CTM dependent, so we cannot take the
drawBitmap() fast path in the presence of mask filters.

BUG=skia:5682
R=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2286873002
==========

to

==========
drawBitmapRect() should not touch the CTM when mask filters are present

Blur sigma calculations are CTM dependent, so we cannot take the
drawBitmap() fast path in the presence of mask filters.

BUG=skia:5682
R=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2286873002

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

[commit-bot: I haz the power, 2016-08-26 20:04:17]

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