Insert an empty image when we cannot decode

Insert an empty image when we cannot decode

In SKP deserialization, if there is a 0 byte encoded image, we insert
an empty image into the output SkPicture so that we can continue
deserializing. Do the same when our decoder cannot decode it. This may
be the result of a bug in our decoder, or in the stream serialization.
But it should not mean the entire stream is invalid.

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

Committed: https://skia.googlesource.com/skia/+/950fb1ca24cca9803eb29dc6aa5472b0f506a67f

[scroggo, 2016-02-19 19:23:37]

Description was changed from

==========
Insert an empty image when we cannot decode

In SKP deserialization, if there is a 0 byte encoded image, we insert
an empty image into the output SkPicture so that we can continue
deserializing. Do the same when our decoder cannot decode it. This may
be the result of a bug in our decoder, or in the stream serialization.
But it should not mean the entire stream is invalid.

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

to

==========
Insert an empty image when we cannot decode

In SKP deserialization, if there is a 0 byte encoded image, we insert
an empty image into the output SkPicture so that we can continue
deserializing. Do the same when our decoder cannot decode it. This may
be the result of a bug in our decoder, or in the stream serialization.
But it should not mean the entire stream is invalid.

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

[scroggo, 2016-02-19 19:25:12]

scroggo@google.com changed reviewers:
+ mtklein@google.com

[mtklein, 2016-02-19 19:30:35]

lgtm

[scroggo, 2016-02-19 19:33:26]

The CQ bit was checked by scroggo@google.com

[commit-bot: I haz the power, 2016-02-19 19:33:30]

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

[commit-bot: I haz the power, 2016-02-19 20:08:43]

Description was changed from

==========
Insert an empty image when we cannot decode

In SKP deserialization, if there is a 0 byte encoded image, we insert
an empty image into the output SkPicture so that we can continue
deserializing. Do the same when our decoder cannot decode it. This may
be the result of a bug in our decoder, or in the stream serialization.
But it should not mean the entire stream is invalid.

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

to

==========
Insert an empty image when we cannot decode

In SKP deserialization, if there is a 0 byte encoded image, we insert
an empty image into the output SkPicture so that we can continue
deserializing. Do the same when our decoder cannot decode it. This may
be the result of a bug in our decoder, or in the stream serialization.
But it should not mean the entire stream is invalid.

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

Committed:
https://skia.googlesource.com/skia/+/950fb1ca24cca9803eb29dc6aa5472b0f506a67f
==========

[commit-bot: I haz the power, 2016-02-19 20:08:44]

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