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

Issue 2684233002: Update re2 fuzzer to follow the upstream version. (Closed)

Created:
3 years, 10 months ago by mmoroz
Modified:
3 years, 10 months ago
Reviewers:
Oliver Chang, inferno
CC:
chromium-reviews, fuzzing_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Update re2 fuzzer to follow the upstream version. BUG= TBR=inferno@chromium.org,ochang@chromium.org Review-Url: https://codereview.chromium.org/2684233002 Cr-Commit-Position: refs/heads/master@{#449257} Committed: https://chromium.googlesource.com/chromium/src/+/706a34d2abe59db6e34b212c1ff2de64ddc7904c

Patch Set 1 #

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

Messages

Total messages: 9 (5 generated)
mmoroz
On 2017/02/09 10:03:58, mmoroz wrote: > mailto:mmoroz@chromium.org changed reviewers: > + mailto:inferno@chromium.org, mailto:ochang@chromium.org This is ...
3 years, 10 months ago (2017-02-09 10:09:03 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2684233002/1
3 years, 10 months ago (2017-02-09 10:09:26 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/src/+/706a34d2abe59db6e34b212c1ff2de64ddc7904c
3 years, 10 months ago (2017-02-09 11:10:43 UTC) #8
inferno
3 years, 10 months ago (2017-02-09 12:16:36 UTC) #9
Message was sent while issue was closed.
lgtm, but ideal way is to roll re2 forward and use fuzzer from that repo
directly.

Powered by Google App Engine
This is Rietveld 408576698