ValueSerializer: Check for no matching ArrayBufferView type being found.

ValueSerializer: Check for no matching ArrayBufferView type being found.

Previously this would result in applying trying to find a size modulo zero,
which causes SIGFPE. This approach was preferred over adding a default case
to preserve the ability of the compiler to detect unhandled switch cases
(within the valid range of the enum).

BUG=chromium:148757
Committed: https://crrev.com/e4cc955780804a98425bc927742bd6a0907ea543
Cr-Commit-Position: refs/heads/master@{#40088}

[jbroman, 2016-10-06 16:48:06]

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

[commit-bot: I haz the power, 2016-10-06 16:48:10]

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

[jbroman, 2016-10-06 16:52:51]

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

[commit-bot: I haz the power, 2016-10-06 16:53:02]

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-10-06 17:16:35]

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

[commit-bot: I haz the power, 2016-10-06 17:16:35]

Dry run: This issue passed the CQ dry run.

[jbroman, 2016-10-06 17:20:31]

jbroman@chromium.org changed reviewers:
+ cbruni@chromium.org, jkummerow@chromium.org

[jbroman, 2016-10-06 17:20:31]

Also found with libfuzzer.

[Jakob Kummerow, 2016-10-07 09:28:23]

lgtm

[jbroman, 2016-10-07 13:11:48]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2016-10-07 13:11:57]

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-10-07 13:14:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-07 13:15:15]

Description was changed from

==========
ValueSerializer: Check for no matching ArrayBufferView type being found.

Previously this would result in applying trying to find a size modulo zero,
which causes SIGFPE. This approach was preferred over adding a default case
to preserve the ability of the compiler to detect unhandled switch cases
(within the valid range of the enum).

BUG=chromium:148757
==========

to

==========
ValueSerializer: Check for no matching ArrayBufferView type being found.

Previously this would result in applying trying to find a size modulo zero,
which causes SIGFPE. This approach was preferred over adding a default case
to preserve the ability of the compiler to detect unhandled switch cases
(within the valid range of the enum).

BUG=chromium:148757

Committed: https://crrev.com/e4cc955780804a98425bc927742bd6a0907ea543
Cr-Commit-Position: refs/heads/master@{#40088}
==========

[commit-bot: I haz the power, 2016-10-07 13:15:16]

Patchset 2 (id:??) landed as
https://crrev.com/e4cc955780804a98425bc927742bd6a0907ea543
Cr-Commit-Position: refs/heads/master@{#40088}