Stop valgrind bot from crashing

Stop valgrind bot from crashing

make_surface is returning nullptr, resulting in an invalid read.

Add an error message with debugging information and skip this
iteration.

Also switch to cleaner for loop syntax.

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

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

[scroggo_chromium, 2016-05-09 19:43:55]

Description was changed from

==========
Stop valgrind bot from crashing

make_surface is returning nullptr, resulting in an invalid read.

Add an error message with debugging information and skip this
iteration.

Also switch to cleaner for loop syntax.

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

to

==========
Stop valgrind bot from crashing

make_surface is returning nullptr, resulting in an invalid read.

Add an error message with debugging information and skip this
iteration.

Also switch to cleaner for loop syntax.

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

[scroggo_chromium, 2016-05-09 19:44:41]

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

[commit-bot: I haz the power, 2016-05-09 19:44:46]

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

[scroggo_chromium, 2016-05-09 19:47:33]

scroggo@chromium.org changed reviewers:
+ egdaniel@google.com

[commit-bot: I haz the power, 2016-05-09 20:10:14]

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

[commit-bot: I haz the power, 2016-05-09 20:10:15]

Dry run: This issue passed the CQ dry run.

[egdaniel, 2016-05-09 20:18:12]

lgtm

[scroggo, 2016-05-09 20:19:53]

The CQ bit was checked by scroggo@google.com

[commit-bot: I haz the power, 2016-05-09 20:20:07]

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

[commit-bot: I haz the power, 2016-05-09 20:21:02]

Description was changed from

==========
Stop valgrind bot from crashing

make_surface is returning nullptr, resulting in an invalid read.

Add an error message with debugging information and skip this
iteration.

Also switch to cleaner for loop syntax.

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

to

==========
Stop valgrind bot from crashing

make_surface is returning nullptr, resulting in an invalid read.

Add an error message with debugging information and skip this
iteration.

Also switch to cleaner for loop syntax.

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

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

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

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