Simplify parse_certificate_fuzzer.

Simplify parse_certificate_fuzzer.

Committed: https://crrev.com/abee8b631557f5d01a728b3d7e6416011cfb85e5
Cr-Commit-Position: refs/heads/master@{#420688}

[eroman, 2016-09-22 07:07:15]

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

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

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

[eroman, 2016-09-22 07:09:48]

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

[commit-bot: I haz the power, 2016-09-22 07:17:49]

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

[commit-bot: I haz the power, 2016-09-22 07:17:50]

Dry run: This issue passed the CQ dry run.

[mattm, 2016-09-22 22:57:14]

lgtm

https://codereview.chromium.org/2366493003/diff/1/net/cert/internal/parse_cer...
File net/cert/internal/parse_certificate_fuzzer.cc (left):

https://codereview.chromium.org/2366493003/diff/1/net/cert/internal/parse_cer...
net/cert/internal/parse_certificate_fuzzer.cc:66:
ParseCertificatePoliciesExtension(extension.value, &policies);
Doesn't fuzz ParseCertificatePoliciesExtension and ParseEKUExtension extension
anymore.. but I guess it's better to fuzz the code path we actually use, and
it'll start fuzzing them again if we add them to ParsedCertificate.

[eroman, 2016-09-22 23:22:23]

https://codereview.chromium.org/2366493003/diff/1/net/cert/internal/parse_cer...
File net/cert/internal/parse_certificate_fuzzer.cc (left):

https://codereview.chromium.org/2366493003/diff/1/net/cert/internal/parse_cer...
net/cert/internal/parse_certificate_fuzzer.cc:66:
ParseCertificatePoliciesExtension(extension.value, &policies);
On 2016/09/22 22:57:14, mattm wrote:
> Doesn't fuzz ParseCertificatePoliciesExtension and ParseEKUExtension extension
> anymore.. but I guess it's better to fuzz the code path we actually use, and
> it'll start fuzzing them again if we add them to ParsedCertificate.

Correct. I expect to add those to ParsedCertificate at some point.

[eroman, 2016-09-23 18:58:25]

The CQ bit was checked by eroman@chromium.org

[eroman, 2016-09-23 18:58:25]

The patchset sent to the CQ was uploaded after l-g-t-m from mattm@chromium.org
Link to the patchset: https://codereview.chromium.org/2366493003/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2016-09-23 18:59:24]

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-23 19:10:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-23 19:12:25]

Description was changed from

==========
Simplify parse_certificate_fuzzer.
==========

to

==========
Simplify parse_certificate_fuzzer.

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

[commit-bot: I haz the power, 2016-09-23 19:12:26]

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