Fix crash when ETC1 codec is not supported.

Fix crash when ETC1 codec is not supported.

BUG=skia:4216
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1567893002

Committed: https://skia.googlesource.com/skia/+/19aa2b08678499a0d9c146bcd409a8f73f5e5d5c

[djsollen, 2016-01-07 14:57:14]

Description was changed from

==========
Fix crash when ETC1 codec is not supported.

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

to

==========
Fix crash when ETC1 codec is not supported.

BUG=skia:4216
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[djsollen, 2016-01-07 14:58:31]

djsollen@google.com changed reviewers:
+ reed@google.com, tomhudson@google.com

[djsollen, 2016-01-07 14:59:19]

https://codereview.chromium.org/1567893002/diff/1/gm/etc1bitmap.cpp
File gm/etc1bitmap.cpp (right):

https://codereview.chromium.org/1567893002/diff/1/gm/etc1bitmap.cpp#newcode108
gm/etc1bitmap.cpp:108: if (image) {
should we print a debug statement if image == nullptr?

[djsollen, 2016-01-07 15:03:28]

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

[commit-bot: I haz the power, 2016-01-07 15:03:37]

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

[tomhudson, 2016-01-07 15:17:34]

On 2016/01/07 14:59:19, djsollen wrote:
> should we print a debug statement if image == nullptr?

If we're going to fail in a way that's detectable by GM (I expect so - no image
drawn), I'd suspect we shouldn't.

But either way LGTM.

[commit-bot: I haz the power, 2016-01-07 15:21:42]

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

[commit-bot: I haz the power, 2016-01-07 15:21:42]

Dry run: This issue passed the CQ dry run.

[djsollen, 2016-01-07 15:22:03]

The CQ bit was checked by djsollen@google.com

[commit-bot: I haz the power, 2016-01-07 15:22:05]

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

[commit-bot: I haz the power, 2016-01-07 15:22:48]

Description was changed from

==========
Fix crash when ETC1 codec is not supported.

BUG=skia:4216
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Fix crash when ETC1 codec is not supported.

BUG=skia:4216
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/19aa2b08678499a0d9c146bcd409a8f73f5e5d5c
==========

[commit-bot: I haz the power, 2016-01-07 15:22:49]

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