Remove LLVMFuzzerInitialize from many Blink fuzzers

Remove LLVMFuzzerInitialize from many Blink fuzzers

This function should only be used if argv is needed, otherwise libfuzzer
best practice is to just use static initialization in
LLVMFuzzerTestOneInput [1].

This CL also changes the BlinkFuzzerTestSupport to be a class which
does all initialization in its constructor.

[1]: http://llvm.org/docs/LibFuzzer.html#startup-initialization

BUG=693760
Review-Url: https://codereview.chromium.org/2900373002
Cr-Commit-Position: refs/heads/master@{#474993}
Committed: https://chromium.googlesource.com/chromium/src/+/fda3f3decbf18cec68fed81b8f48add43c3c64b2

[Charlie Harrison, 2017-05-24 19:11:47]

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

[commit-bot: I haz the power, 2017-05-24 19:12:54]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Charlie Harrison, 2017-05-24 19:51:12]

csharrison@chromium.org changed reviewers:
+ mmoroz@chromium.org

[Charlie Harrison, 2017-05-24 19:51:13]

mmoroz would you take a first look? I double checked two of these fuzzers
manually but as usual this kind of mass fuzzer change makes me nervous.

[commit-bot: I haz the power, 2017-05-24 20:53:38]

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

[commit-bot: I haz the power, 2017-05-24 20:53:39]

Dry run: This issue passed the CQ dry run.

[Charlie Harrison, 2017-05-25 03:59:25]

csharrison@chromium.org changed reviewers:
+ haraken@chromium.org

[Charlie Harrison, 2017-05-25 03:59:26]

+haraken as well, LMK if you would prefer more specific owners.

[haraken, 2017-05-25 04:10:44]

LGTM

[mmoroz, 2017-05-26 06:09:50]

LGTM

[Charlie Harrison, 2017-05-26 12:21:44]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2017-05-26 12:22:03]

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, 2017-05-26 13:35:15]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495801304723490, "parent_rev":
"f45dbb0e42abd0c94f144ecb6966893a17759c07", "commit_rev":
"fda3f3decbf18cec68fed81b8f48add43c3c64b2"}

[commit-bot: I haz the power, 2017-05-26 13:35:28]

Description was changed from

==========
Remove LLVMFuzzerInitialize from many Blink fuzzers

This function should only be used if argv is needed, otherwise libfuzzer
best practice is to just use static initialization in
LLVMFuzzerTestOneInput [1].

This CL also changes the BlinkFuzzerTestSupport to be a class which
does all initialization in its constructor.

[1]: http://llvm.org/docs/LibFuzzer.html#startup-initialization

BUG=693760
==========

to

==========
Remove LLVMFuzzerInitialize from many Blink fuzzers

This function should only be used if argv is needed, otherwise libfuzzer
best practice is to just use static initialization in
LLVMFuzzerTestOneInput [1].

This CL also changes the BlinkFuzzerTestSupport to be a class which
does all initialization in its constructor.

[1]: http://llvm.org/docs/LibFuzzer.html#startup-initialization

BUG=693760

Review-Url: https://codereview.chromium.org/2900373002
Cr-Commit-Position: refs/heads/master@{#474993}
Committed:
https://chromium.googlesource.com/chromium/src/+/fda3f3decbf18cec68fed81b8f48...
==========

[commit-bot: I haz the power, 2017-05-26 13:35:32]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/fda3f3decbf18cec68fed81b8f48...