Remove isOpaque from SkSpecialImage, just use alpha type

Remove isOpaque from SkSpecialImage, just use alpha type

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

Committed: https://skia.googlesource.com/skia/+/80e96088bc608d6935b24ff623b9025b38da576e

[Brian Osman, 2016-08-15 20:44:23]

Description was changed from

==========
Remove isOpaque from SkSpecialImage, just use alpha type

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

to

==========
Remove isOpaque from SkSpecialImage, just use alpha type

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

[Brian Osman, 2016-08-15 20:44:35]

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

[commit-bot: I haz the power, 2016-08-15 20:44:40]

Dry run: 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-15 21:09:06]

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

[commit-bot: I haz the power, 2016-08-15 21:09:07]

Dry run: This issue passed the CQ dry run.

[Brian Osman, 2016-08-15 21:18:59]

brianosman@google.com changed reviewers:
+ bsalomon@google.com, robertphillips@google.com

[Brian Osman, 2016-08-15 21:18:59]

We have isOpaque() on other types, too (including SkImage). It's not clear to me
why we narrow the information we store/return like that (vs. just reporting
alphaType). Does this make sense? I think making this change (and similar
changes elsewhere) makes things strictly simpler.

[Brian Osman, 2016-08-16 13:23:14]

... pure virtual.

[robertphillips, 2016-08-16 13:26:55]

lgtm

[robertphillips, 2016-08-16 13:27:59]

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

[commit-bot: I haz the power, 2016-08-16 13:28:02]

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

[Brian Osman, 2016-08-16 13:40:31]

The CQ bit was unchecked by brianosman@google.com

[Brian Osman, 2016-08-16 13:40:36]

The CQ bit was checked by brianosman@google.com

[commit-bot: I haz the power, 2016-08-16 13:40:46]

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

[bsalomon, 2016-08-16 13:52:36]

On 2016/08/16 13:40:46, commit-bot: I haz the power wrote:
> 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-16 14:09:49]

Description was changed from

==========
Remove isOpaque from SkSpecialImage, just use alpha type

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

to

==========
Remove isOpaque from SkSpecialImage, just use alpha type

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

Committed:
https://skia.googlesource.com/skia/+/80e96088bc608d6935b24ff623b9025b38da576e
==========

[commit-bot: I haz the power, 2016-08-16 14:09:50]

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