|
|
Created:
6 years, 2 months ago by Yoav Weiss Modified:
6 years, 2 months ago CC:
chromium-reviews, asvitkine+watch_chromium.org Base URL:
https://chromium.googlesource.com/chromium/src.git@master Project:
chromium Visibility:
Public. |
DescriptionAdded histograms for newly added use counters.
Following https://codereview.chromium.org/649183007/ , I've updated the
histograms file on the Chrome side.
Committed: https://crrev.com/f6495730cb973fb920701f04af53ee635bf44f38
Cr-Commit-Position: refs/heads/master@{#300638}
Patch Set 1 #Patch Set 2 : rebase #
Messages
Total messages: 20 (8 generated)
yoav@yoav.ws changed reviewers: + mkwst@chromium.org
mkwst@chromium.org changed reviewers: + isherman@chromium.org
1. This doesn't seem to apply cleanly. I assume one or the other item has already been added on ToT. 2. +Ilya to stamp the CL.
On 2014/10/21 12:41:59, Mike West wrote: > 1. This doesn't seem to apply cleanly. I assume one or the other item has > already been added on ToT. Yeah, rebased now after manually git pulling (my sync scripts already include git pulling, but they seem to be overridden by "gclient sync". Oh well) > 2. +Ilya to stamp the CL.
mkwst@chromium.org changed required reviewers: + isherman@chromium.org
LGTM, but you'll need Ilya.
LGTM
The CQ bit was checked by yoav@yoav.ws
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/663393002/20001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: win_gpu_triggered_tests on tryserver.chromium.gpu (http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)
The CQ bit was checked by yoav@yoav.ws
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/663393002/20001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: win_gpu_triggered_tests on tryserver.chromium.gpu (http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)
The CQ bit was checked by yoav@yoav.ws
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/663393002/20001
Message was sent while issue was closed.
Committed patchset #2 (id:20001)
Message was sent while issue was closed.
Patchset 2 (id:??) landed as https://crrev.com/f6495730cb973fb920701f04af53ee635bf44f38 Cr-Commit-Position: refs/heads/master@{#300638} |