Fix HttpStreamParser fuzzer.

Fix HttpStreamParser fuzzer.

Reading HTTP headers returns net::OK on some success paths, and bytes
read on others.  The test didn't account for this.  This CL fixes the
fuzzer to account for that.  The weird HttpStreamParser behavior should
also be fixed.

BUG=598037
Committed: https://crrev.com/f344e9f0258f8d9f79b97f8ff445bb0d6fed1789
Cr-Commit-Position: refs/heads/master@{#386197}

[mmenke, 2016-04-08 20:19:38]

mmenke@chromium.org changed reviewers:
+ eroman@chromium.org

[mmenke, 2016-04-08 20:19:39]

Grumble...If only it were easy to run and debug these on Windows, I would have
noticed this earlier.  I have no verified it works on Windows with a debugger.

Note that sync header read successes do return net::OK, and so do some other
paths, so we weren't completely missing coverage, but...ick.

[eroman, 2016-04-08 20:58:59]

lgtm

[mmenke, 2016-04-08 20:59:55]

On 2016/04/08 20:58:59, eroman wrote:
> lgtm

Thanks!  Good thing fuzzers report code coverage, or I never would have found
this.

[mmenke, 2016-04-08 21:00:04]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2016-04-08 21:00:38]

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

[commit-bot: I haz the power, 2016-04-08 21:06:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-08 21:08:23]

Description was changed from

==========
Fix HttpStreamParser fuzzer.

Reading HTTP headers returns net::OK on some success paths, and bytes
read on others.  The test didn't account for this.  This CL fixes the
fuzzer to account for that.  The weird HttpStreamParser behavior should
also be fixed.

BUG=598037
==========

to

==========
Fix HttpStreamParser fuzzer.

Reading HTTP headers returns net::OK on some success paths, and bytes
read on others.  The test didn't account for this.  This CL fixes the
fuzzer to account for that.  The weird HttpStreamParser behavior should
also be fixed.

BUG=598037

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

[commit-bot: I haz the power, 2016-04-08 21:08:24]

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