Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(174)

Issue 1703033002: [libfuzzer] disable v8_script_parser_fuzzer, because it is useless at the moment. (Closed)

Created:
4 years, 10 months ago by mmoroz
Modified:
4 years, 10 months ago
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[libfuzzer] disable v8_script_parser_fuzzer, because it is useless at the moment. v8_script_parser_fuzzer fails for two reasons atm: 1) too large corpus -> need to implement corpus pruning 2) strange timeouts ("ALARM: working on the last Unit for 1455099776 seconds") R=inferno@chromium.org, kcc@chromium.org BUG=539572 Committed: https://crrev.com/0e477e65ba0cff13a60b871aae3af0b95dee43d6 Cr-Commit-Position: refs/heads/master@{#375900}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M testing/libfuzzer/fuzzers/BUILD.gn View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 30 (14 generated)
mmoroz
4 years, 10 months ago (2016-02-17 15:21:01 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1703033002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1703033002/1
4 years, 10 months ago (2016-02-17 15:23:06 UTC) #4
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. Only full committers are accepted. Even if an ...
4 years, 10 months ago (2016-02-17 15:23:07 UTC) #6
kcc2
lgtm 2) is the same as 1) -- hapens when libFuzzer is loading the corpus ...
4 years, 10 months ago (2016-02-17 15:23:45 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1703033002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1703033002/1
4 years, 10 months ago (2016-02-17 15:23:52 UTC) #9
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. Only full committers are accepted. Even if an ...
4 years, 10 months ago (2016-02-17 15:23:54 UTC) #11
mmoroz
On 2016/02/17 15:21:01, mmoroz wrote: I let myself to put 'TBR' and click Commit, because ...
4 years, 10 months ago (2016-02-17 15:24:20 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1703033002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1703033002/1
4 years, 10 months ago (2016-02-17 15:25:06 UTC) #14
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. Only full committers are accepted. Even if an ...
4 years, 10 months ago (2016-02-17 15:25:07 UTC) #16
mmoroz
On 2016/02/17 15:25:07, commit-bot: I haz the power wrote: > No L-G-T-M from a valid ...
4 years, 10 months ago (2016-02-17 15:26:47 UTC) #17
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1703033002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1703033002/1
4 years, 10 months ago (2016-02-17 15:27:49 UTC) #20
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. Only full committers are accepted. Even if an ...
4 years, 10 months ago (2016-02-17 15:27:52 UTC) #22
jochen (gone - plz use gerrit)
lgtm
4 years, 10 months ago (2016-02-17 15:30:15 UTC) #24
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1703033002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1703033002/1
4 years, 10 months ago (2016-02-17 15:31:03 UTC) #26
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 10 months ago (2016-02-17 16:23:41 UTC) #28
commit-bot: I haz the power
4 years, 10 months ago (2016-02-17 16:24:57 UTC) #30
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/0e477e65ba0cff13a60b871aae3af0b95dee43d6
Cr-Commit-Position: refs/heads/master@{#375900}

Powered by Google App Engine
This is Rietveld 408576698