|
|
Chromium Code Reviews|
Created:
4 years, 6 months ago by François Beaufort Modified:
4 years, 5 months ago CC:
asvitkine+watch_chromium.org, chromium-reviews Base URL:
https://chromium.googlesource.com/chromium/src.git@master Target Ref:
refs/pending/heads/master Project:
chromium Visibility:
Public. |
Descriptionbluetooth: Enable UMA tracking of --enable-web-bluetooth flag
Committed: https://crrev.com/f23214897b2c3674808544906b6cc7eb49ca9d25
Cr-Commit-Position: refs/heads/master@{#403703}
Patch Set 1 #
Messages
Total messages: 21 (8 generated)
Description was changed from ========== bluetooth: Enable UMA tracking of --enable-web-bluetooth flag BUG=622495 ========== to ========== bluetooth: Enable UMA tracking of --enable-web-bluetooth flag ==========
beaufort.francois@gmail.com changed reviewers: + ortuno@chromium.org
Hello Gio, In the same vein as crbug.com/622495, I think this would be valuable for us to get a sense of how often users run with experimental Web Bluetooth enabled in chrome://flags.
As mentioned offline we already histogram usage of the --enable-web-bluetooth flag.
ortuno@chromium.org changed reviewers: + scheib@chromium.org
scheib: can you take a look at this?
On 2016/06/24 20:37:49, ortuno wrote: > scheib: can you take a look at this? FYI, it looks like the existing UMA for Web Bluetooth flag works only for Chrome OS. I'm not able to see the ones from Android, Linux and macOS.
On 2016/06/27 12:05:05, François Beaufort wrote: > On 2016/06/24 20:37:49, ortuno wrote: > > scheib: can you take a look at this? > > FYI, it looks like the existing UMA for Web Bluetooth flag works only for Chrome > OS. > I'm not able to see the ones from Android, Linux and macOS. @scheib (gentle ping)
LGTM
The CQ bit was checked by beaufort.francois@gmail.com
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
beaufort.francois@gmail.com changed reviewers: + rkaplow@chromium.org
rkaplow, please can you review changes in tools/metrics/actions/actions.xml Thank you!
lgtm
The CQ bit was checked by beaufort.francois@gmail.com
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== bluetooth: Enable UMA tracking of --enable-web-bluetooth flag ========== to ========== bluetooth: Enable UMA tracking of --enable-web-bluetooth flag Committed: https://crrev.com/f23214897b2c3674808544906b6cc7eb49ca9d25 Cr-Commit-Position: refs/heads/master@{#403703} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/f23214897b2c3674808544906b6cc7eb49ca9d25 Cr-Commit-Position: refs/heads/master@{#403703} |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
