Detect an invalid intervalCount in SkRegion during deserialiation.

Detect an invalid intervalCount in SkRegion during deserialiation.

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

Committed: https://skia.googlesource.com/skia/+/675576f023c8fa10cdb0c18bc0a6c214e0bab069

[Stephen White, 2016-05-06 15:23:07]

Description was changed from

==========
Detect an invalid intervalCount in SkRegion during deserialiation.

BUG=609260
==========

to

==========
Detect an invalid intervalCount in SkRegion during deserialiation.

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

[Stephen White, 2016-05-06 15:23:32]

Description was changed from

==========
Detect an invalid intervalCount in SkRegion during deserialiation.

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

to

==========
Detect an invalid intervalCount in SkRegion during deserialiation.

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

[Stephen White, 2016-05-06 15:23:32]

senorblanco@chromium.org changed reviewers:
+ robertphillips@google.com

[Stephen White, 2016-05-06 15:23:40]

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

[commit-bot: I haz the power, 2016-05-06 15:23:41]

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

[Stephen White, 2016-05-06 15:23:51]

Rob: PTAL. Thanks!

[robertphillips, 2016-05-06 15:38:35]

lgtm

[commit-bot: I haz the power, 2016-05-06 15:47:32]

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

[commit-bot: I haz the power, 2016-05-06 15:47:33]

Dry run: This issue passed the CQ dry run.

[Stephen White, 2016-05-06 15:47:57]

The CQ bit was checked by senorblanco@chromium.org

[commit-bot: I haz the power, 2016-05-06 15:48:02]

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

[commit-bot: I haz the power, 2016-05-06 15:48:59]

Description was changed from

==========
Detect an invalid intervalCount in SkRegion during deserialiation.

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

to

==========
Detect an invalid intervalCount in SkRegion during deserialiation.

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

Committed:
https://skia.googlesource.com/skia/+/675576f023c8fa10cdb0c18bc0a6c214e0bab069
==========

[commit-bot: I haz the power, 2016-05-06 15:49:00]

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

[reed1, 2016-05-06 17:00:24]

reed@google.com changed reviewers:
+ reed@google.com

[reed1, 2016-05-06 17:00:25]

1. comment why intervalCount of 1 is not valid?
2. comment why we don't check ySpanCount?
3. is it reasonable to create a unittest or fuzzer to exercise this?