implement isABitmap for imageshader, return localmatrix for bitmap's impl

implement isABitmap for imageshader, return localmatrix for bitmap's impl

For imageshader, I only return true if the image is explicitly raster-backed. I do not return true for texture, nor for generator (i.e. lazy/picture) backed. Is that ok?

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

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

[reed1, 2016-08-01 17:28:30]

Description was changed from

==========
implement isABitmap for imageshader, return localmatrix for bitmap's impl

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

to

==========
implement isABitmap for imageshader, return localmatrix for bitmap's impl

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

[reed1, 2016-08-01 17:28:51]

reed@google.com changed reviewers:
+ djsollen@google.com, halcanary@google.com, mtklein@google.com

[reed1, 2016-08-01 17:28:55]

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

[commit-bot: I haz the power, 2016-08-01 17:28:57]

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-01 17:31:30]

The CQ bit was unchecked by reed@google.com

[reed1, 2016-08-01 17:31:33]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2016-08-01 17:31:37]

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-01 17:39:06]

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

[commit-bot: I haz the power, 2016-08-01 17:39:07]

Try jobs failed on following builders:
  Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-Trybot on master.client.skia
(JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2...)

[reed1, 2016-08-01 17:40:08]

update pdf code

[reed1, 2016-08-01 17:40:13]

The CQ bit was checked by reed@google.com

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

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

[reed1, 2016-08-01 17:40:51]

Description was changed from

==========
implement isABitmap for imageshader, return localmatrix for bitmap's impl

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

to

==========
implement isABitmap for imageshader, return localmatrix for bitmap's impl

For imageshader, I only return true if the image is explicitly raster-backed. I
do not return true for texture, nor for generator (i.e. lazy/picture) backed. Is
that ok?

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

[reed1, 2016-08-01 17:41:39]

The CQ bit was unchecked by reed@google.com

[reed1, 2016-08-01 17:55:34]

now with isAImage api

[reed1, 2016-08-01 17:55:37]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2016-08-01 17:55:52]

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

[hal.canary, 2016-08-01 18:03:31]

lgtm

[commit-bot: I haz the power, 2016-08-01 18:24:16]

Description was changed from

==========
implement isABitmap for imageshader, return localmatrix for bitmap's impl

For imageshader, I only return true if the image is explicitly raster-backed. I
do not return true for texture, nor for generator (i.e. lazy/picture) backed. Is
that ok?

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

to

==========
implement isABitmap for imageshader, return localmatrix for bitmap's impl

For imageshader, I only return true if the image is explicitly raster-backed. I
do not return true for texture, nor for generator (i.e. lazy/picture) backed. Is
that ok?

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

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

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

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/f1ac18229c3f0f813a128a10111766c991f9dfcf