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

Issue 1529203003: Add DrFuzz support to webrtc fuzzers. (Closed)

Created:
5 years ago by pbos-webrtc
Modified:
5 years ago
Reviewers:
kjellander_webrtc
CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com, qiang.lu, niklas.enbom, yujie_mao (webrtc), peah-webrtc, zhaoqin1
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Add DrFuzz support to webrtc fuzzers. BUG=webrtc:4771 R=kjellander@webrtc.org Committed: https://crrev.com/3514cbe554cf510042395a2de35ad3fead670a06 Cr-Commit-Position: refs/heads/master@{#11059}

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -6 lines) Patch
M webrtc/BUILD.gn View 1 chunk +1 line, -1 line 0 comments Download
M webrtc/test/fuzzers/webrtc_fuzzer_main.cc View 1 chunk +3 lines, -5 lines 2 comments Download

Messages

Total messages: 9 (2 generated)
pbos-webrtc
PTAL
5 years ago (2015-12-16 20:07:39 UTC) #1
kjellander_webrtc
lgtm, does this mean we need DrFuzz bots on Windows as well? https://codereview.webrtc.org/1529203003/diff/1/webrtc/test/fuzzers/webrtc_fuzzer_main.cc File webrtc/test/fuzzers/webrtc_fuzzer_main.cc ...
5 years ago (2015-12-16 22:10:26 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1529203003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1529203003/1
5 years ago (2015-12-16 22:12:44 UTC) #4
pbos-webrtc
https://codereview.webrtc.org/1529203003/diff/1/webrtc/test/fuzzers/webrtc_fuzzer_main.cc File webrtc/test/fuzzers/webrtc_fuzzer_main.cc (right): https://codereview.webrtc.org/1529203003/diff/1/webrtc/test/fuzzers/webrtc_fuzzer_main.cc#newcode12 webrtc/test/fuzzers/webrtc_fuzzer_main.cc:12: // It's intended to set sane defaults, such as ...
5 years ago (2015-12-16 22:13:20 UTC) #5
pbos-webrtc
Don't think we're likely to benefit from drfuzz bots since they'd only be the compile ...
5 years ago (2015-12-16 22:18:38 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years ago (2015-12-17 02:36:18 UTC) #7
commit-bot: I haz the power
5 years ago (2015-12-17 02:36:28 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/3514cbe554cf510042395a2de35ad3fead670a06
Cr-Commit-Position: refs/heads/master@{#11059}

Powered by Google App Engine
This is Rietveld 408576698