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

Issue 559583004: Add Owners for HttpServerPropertiesManager-based histograms. (Closed)

Created:
6 years, 3 months ago by cbentzel
Modified:
6 years, 3 months ago
CC:
chromium-reviews, asvitkine+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Add Owners for HttpServerPropertiesManager-based histograms. Committed: https://crrev.com/be98fa8b4452fb3c2ec04e914e0af79d7aca5dd8 Cr-Commit-Position: refs/heads/master@{#293974}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -3 lines) Patch
M tools/metrics/histograms/histograms.xml View 2 chunks +9 lines, -3 lines 0 comments Download

Messages

Total messages: 15 (5 generated)
cbentzel
You seem like good owners for this - but it's worth looking at whether we ...
6 years, 3 months ago (2014-09-09 16:46:18 UTC) #2
Ryan Hamilton
lgtm
6 years, 3 months ago (2014-09-09 16:52:35 UTC) #3
Bence
lgtm
6 years, 3 months ago (2014-09-09 17:11:35 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/cbentzel@chromium.org/559583004/1
6 years, 3 months ago (2014-09-09 17:13:35 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: chromium_presubmit on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presubmit/builds/9683)
6 years, 3 months ago (2014-09-09 17:23:39 UTC) #8
cbentzel
Adding tools/metrics OWNER
6 years, 3 months ago (2014-09-09 17:24:01 UTC) #10
Alexei Svitkine (slow)
lgtm
6 years, 3 months ago (2014-09-09 17:25:04 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/cbentzel@chromium.org/559583004/1
6 years, 3 months ago (2014-09-09 17:26:07 UTC) #13
commit-bot: I haz the power
Committed patchset #1 (id:1) as 71847b761b690cdbbd617c9c30364424727e45dc
6 years, 3 months ago (2014-09-09 18:43:53 UTC) #14
commit-bot: I haz the power
6 years, 3 months ago (2014-09-10 03:54:30 UTC) #15
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/be98fa8b4452fb3c2ec04e914e0af79d7aca5dd8
Cr-Commit-Position: refs/heads/master@{#293974}

Powered by Google App Engine
This is Rietveld 408576698