Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
1858123002:
Updating histogram owners for team changes. (Closed)
|
Created:
4 years, 8 months ago by bradnelson Modified:
4 years, 6 months ago CC:
chromium-reviews, asvitkine+watch_chromium.org Base URL:
https://chromium.googlesource.com/chromium/src.git@master Target Ref:
refs/pending/heads/master Project:
chromium Visibility:
Public. |
DescriptionUpdating histogram owners for team changes.
BUG=None
TEST=None
R=bbudge@chromium.org,isherman@chromium.org
Committed: https://crrev.com/f2365b256a391aa7a44a2a1646205c81c4e30c68
Cr-Commit-Position: refs/heads/master@{#397903}
Patch Set 1 #
Messages
Total messages: 17 (6 generated)
bradnelson@google.com changed reviewers: + bradnelson@google.com
They're apparently about to do a cleanup pass, so figure we better have current owners. Bill, there's a few other ppapi ones that have raymes or raymes + jww, is that still a good set of folks for those?
On 2016/04/04 23:41:36, bradn wrote: > They're apparently about to do a cleanup pass, so figure we better have current > owners. > Bill, there's a few other ppapi ones that have raymes or raymes + jww, is that > still a good set of folks for those? I'd assume those are still valid.
ping?
Description was changed from ========== Updating histogram owners for team changes. BUG=None TEST=None R=bbudge@chromium.org ========== to ========== Updating histogram owners for team changes. BUG=None TEST=None R=bbudge@chromium.org,isherman@chromium.org ==========
Ah, actually maybe not Bill. +isherman Ilya, can you PTAL? Thanks!
bradnelson@chromium.org changed reviewers: + isherman@chromium.org
+ilya PTAL
lgtm
Also need ilya for owners On Jun 3, 2016 5:20 PM, <bbudge@chromium.org> wrote: > lgtm > > > > https://codereview.chromium.org/1858123002/ > -- You received this message because you are subscribed to the Google Groups "Chromium-reviews" group. To unsubscribe from this group and stop receiving emails from it, send an email to chromium-reviews+unsubscribe@chromium.org.
LGTM
The CQ bit was checked by isherman@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1858123002/1
Message was sent while issue was closed.
Description was changed from ========== Updating histogram owners for team changes. BUG=None TEST=None R=bbudge@chromium.org,isherman@chromium.org ========== to ========== Updating histogram owners for team changes. BUG=None TEST=None R=bbudge@chromium.org,isherman@chromium.org ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== Updating histogram owners for team changes. BUG=None TEST=None R=bbudge@chromium.org,isherman@chromium.org ========== to ========== Updating histogram owners for team changes. BUG=None TEST=None R=bbudge@chromium.org,isherman@chromium.org Committed: https://crrev.com/f2365b256a391aa7a44a2a1646205c81c4e30c68 Cr-Commit-Position: refs/heads/master@{#397903} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/f2365b256a391aa7a44a2a1646205c81c4e30c68 Cr-Commit-Position: refs/heads/master@{#397903} |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
