Add a fuzzer for the Origin Trial token parser

Add a fuzzer for the Origin Trial token parser

R=ochang@chromium.org, chasej@chromium.org

Committed: https://crrev.com/62f3ef4fc3059811d5562767d5b1c6b19e21d966
Cr-Commit-Position: refs/heads/master@{#424788}

[iclelland, 2016-09-29 20:14:29]

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

[commit-bot: I haz the power, 2016-09-29 20:15:00]

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-09-29 22:16:47]

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

[commit-bot: I haz the power, 2016-09-29 22:16:48]

Dry run: Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[iclelland, 2016-09-30 15:32:52]

+r chasej, ochang; can you PTAL? Thanks!

ochang: I'm not certain that the way I've handled the seed corpus is the best
way; should there be more samples? fewer? More diverse? Does it even matter?
Expert guidance is much appreciated here.

I also found that the ICU component is required, since this indirectly fuzzes
GURL, so I shamelessly stole that from
testing/libfuzzer/fuzzers/url_parse_fuzzer.cc.

[chasej, 2016-09-30 20:53:40]

lgtm

[Oliver Chang, 2016-10-03 23:35:34]

On 2016/09/30 15:32:52, iclelland wrote:
> +r chasej, ochang; can you PTAL? Thanks!
> 
> ochang: I'm not certain that the way I've handled the seed corpus is the best
> way; should there be more samples? fewer? More diverse? Does it even matter?
> Expert guidance is much appreciated here.
> 
> I also found that the ICU component is required, since this indirectly fuzzes
> GURL, so I shamelessly stole that from
> testing/libfuzzer/fuzzers/url_parse_fuzzer.cc.

lgtm.

The seed corpus seems reasonable, but it's hard to say how effective it is
without actually doing experiments by running the fuzzer :)

[iclelland, 2016-10-04 04:04:39]

The CQ bit was checked by iclelland@chromium.org

[commit-bot: I haz the power, 2016-10-04 04:04:58]

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-04 04:13:59]

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

[commit-bot: I haz the power, 2016-10-04 04:14:00]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[iclelland, 2016-10-04 04:27:14]

iclelland@chromium.org changed reviewers:
+ aizatsky@chromium.org

[iclelland, 2016-10-04 04:27:15]

+r aizatsky -- can you PTAL as owner of content/test/fuzzer? Thanks!

[iclelland, 2016-10-12 04:19:45]

On 2016/10/04 04:27:15, iclelland wrote:
> +r aizatsky -- can you PTAL as owner of content/test/fuzzer? Thanks!

aizatsky - ping :)

[aizatsky, 2016-10-12 16:58:59]

lgtm

[iclelland, 2016-10-12 17:18:31]

The CQ bit was checked by iclelland@chromium.org

[commit-bot: I haz the power, 2016-10-12 17:19:05]

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-12 17:28:09]

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

[commit-bot: I haz the power, 2016-10-12 17:28:10]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)

[iclelland, 2016-10-12 17:47:40]

The CQ bit was checked by iclelland@chromium.org

[commit-bot: I haz the power, 2016-10-12 17:48:08]

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-12 18:23:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-12 18:26:26]

Description was changed from

==========
Add a fuzzer for the Origin Trial token parser

R=ochang@chromium.org, chasej@chromium.org
==========

to

==========
Add a fuzzer for the Origin Trial token parser

R=ochang@chromium.org, chasej@chromium.org

Committed: https://crrev.com/62f3ef4fc3059811d5562767d5b1c6b19e21d966
Cr-Commit-Position: refs/heads/master@{#424788}
==========

[commit-bot: I haz the power, 2016-10-12 18:26:27]

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