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

Issue 1695803002: Improve fuzz counts (Closed)

Created:
4 years, 10 months ago by kjlubick
Modified:
4 years, 10 months ago
Reviewers:
jcgregorio
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/buildbot@dedup-first
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Improve fuzz counts This replaces the "Always scan through all of the GCS files every N minutes to count fuzzes" with something that 1) caches results of things that won't change and 2) is correct. The previous fuzz counts were often wrong (what does -34 new bad fuzzes mean?) and were slow. This also includes a few expand-related UI glitch fixes and replaces the "new grey fuzz" count with "new regressions", which should be more useful. BUG=skia:4438 Committed: https://skia.googlesource.com/buildbot/+/c14a8420c8a6a72b64332a3ec88874472ba1bc57

Patch Set 1 #

Patch Set 2 : rebasing #

Patch Set 3 : Add docs #

Unified diffs Side-by-side diffs Delta from patch set Stats (+67 lines, -39 lines) Patch
M fuzzer/go/data/report.go View 3 chunks +3 lines, -0 lines 0 comments Download
M fuzzer/go/frontend/syncer/fuzz_syncer.go View 1 2 6 chunks +53 lines, -28 lines 0 comments Download
M fuzzer/go/fuzzer-fe/main.go View 3 chunks +7 lines, -7 lines 0 comments Download
M fuzzer/res/imp/fuzzer-collapse-file-sk.html View 1 chunk +1 line, -1 line 0 comments Download
M fuzzer/res/imp/fuzzer-filter-sk.html View 1 chunk +1 line, -1 line 0 comments Download
M fuzzer/res/imp/fuzzer-info-sk.html View 1 chunk +1 line, -1 line 0 comments Download
M fuzzer/res/imp/fuzzer-summary-sk.html View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 12 (6 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1695803002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1695803002/40001
4 years, 10 months ago (2016-02-12 20:58:29 UTC) #3
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 10 months ago (2016-02-12 21:01:36 UTC) #5
kjlubick
4 years, 10 months ago (2016-02-12 21:10:09 UTC) #7
jcgregorio
lgtm
4 years, 10 months ago (2016-02-17 14:42:52 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1695803002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1695803002/40001
4 years, 10 months ago (2016-02-17 14:45:26 UTC) #10
commit-bot: I haz the power
4 years, 10 months ago (2016-02-17 14:48:37 UTC) #12
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://skia.googlesource.com/buildbot/+/c14a8420c8a6a72b64332a3ec88874472ba1...

Powered by Google App Engine
This is Rietveld 408576698