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

Issue 121763002: Add stats script. (Closed)

Created:
6 years, 12 months ago by cmp
Modified:
6 years, 12 months ago
Reviewers:
eseidel
CC:
chromium-reviews, cmp-cc_chromium.org, ilevy-cc_chromium.org
Visibility:
Public.

Description

Add stats script. This includes some feature additions and bug fixes: - Add additional date modifiers. - Handle case where patchset is missing from an issue. - Fix date issue that occurs with some CLs. - Add support for importing 'stats' and calling its methods. - Add BERSERK and BINARY_FILE2 message handling. - Add FAILED_PATCHSET and INTERNAL_ERROR message handling. - Add APPLY2 message handling. - Change number of items per search to 100. - Make output unbuffered. - Change the QPS throttle to allow up to 20 QPS. Also some extra changes: - Add script to call stats.py. - Ensure all lines are < 80 chars and indent is +2. - Add skia tracking. Original author: eseidel@chromium.org Modifications by: ilevy@chromium.org Further modifications by: cmp@chromium.org TBR=eseidel@chromium.org BUG=330869 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=242625

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+527 lines, -0 lines) Patch
A tools/gather_stats.sh View 1 chunk +23 lines, -0 lines 0 comments Download
A tools/stats.py View 1 chunk +504 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
cmp
6 years, 12 months ago (2013-12-27 18:06:10 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/cmp@chromium.org/121763002/1
6 years, 12 months ago (2013-12-27 18:06:34 UTC) #2
commit-bot: I haz the power
Presubmit check for 121763002-1 failed and returned exit status 1. Running presubmit commit checks ...
6 years, 12 months ago (2013-12-27 18:06:55 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/cmp@chromium.org/121763002/1
6 years, 12 months ago (2013-12-27 18:13:07 UTC) #4
commit-bot: I haz the power
Change committed as 242625
6 years, 12 months ago (2013-12-27 18:13:28 UTC) #5
eseidel
6 years, 12 months ago (2013-12-27 18:15:27 UTC) #6
Message was sent while issue was closed.
Thanks!  Glad this was useful!

Powered by Google App Engine
This is Rietveld 408576698