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

Issue 2922493002: Default to new metrics service URL for non-Cast clients. (Closed)

Created:
3 years, 6 months ago by Alexei Svitkine (slow)
Modified:
3 years, 6 months ago
Reviewers:
rkaplow
CC:
chromium-reviews, alokp+watch_chromium.org, lcwu+watch_chromium.org, asvitkine+watch_chromium.org, halliwell+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Default to new metrics service URL for non-Cast clients. M59-M60 are controlled by the feature, while M61+ will use the new URL exclusively, except for Cast. BUG=703835 Review-Url: https://codereview.chromium.org/2922493002 Cr-Commit-Position: refs/heads/master@{#476407} Committed: https://chromium.googlesource.com/chromium/src/+/ed475699c309203cf9294a75fcc6c9e1a179fd5c

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -38 lines) Patch
M chromecast/browser/metrics/OWNERS View 1 chunk +1 line, -3 lines 0 comments Download
M chromecast/browser/metrics/cast_metrics_service_client.cc View 1 chunk +3 lines, -1 line 0 comments Download
M components/metrics/metrics_service_client.cc View 2 chunks +1 line, -16 lines 0 comments Download
M testing/variations/fieldtrial_testing_config.json View 1 chunk +0 lines, -18 lines 0 comments Download

Messages

Total messages: 11 (7 generated)
Alexei Svitkine (slow)
3 years, 6 months ago (2017-06-01 17:45:36 UTC) #4
rkaplow
lgtm
3 years, 6 months ago (2017-06-01 19:38:39 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2922493002/1
3 years, 6 months ago (2017-06-01 19:41:55 UTC) #8
commit-bot: I haz the power
3 years, 6 months ago (2017-06-01 20:14:45 UTC) #11
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ed475699c309203cf9294a75fcc6...

Powered by Google App Engine
This is Rietveld 408576698