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

Unified Diff: chrome/browser/metrics/chrome_metrics_services_manager_client.h

Issue 2302043002: Adding local field trial for metrics/crash reports sampling. (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/metrics/chrome_metrics_services_manager_client.h
diff --git a/chrome/browser/metrics/chrome_metrics_services_manager_client.h b/chrome/browser/metrics/chrome_metrics_services_manager_client.h
index 7106e1ff7843713b2a43680a9409e5e8667e7386..c3fd6c659177213ec12d463fdf75eb1993acab21 100644
--- a/chrome/browser/metrics/chrome_metrics_services_manager_client.h
+++ b/chrome/browser/metrics/chrome_metrics_services_manager_client.h
@@ -27,6 +27,14 @@ class ChromeMetricsServicesManagerClient
explicit ChromeMetricsServicesManagerClient(PrefService* local_state);
~ChromeMetricsServicesManagerClient() override;
+ // Unconditionally attempts to create a field trial to control client side
+ // metrics/crash sampling to use as a fallback when one hasn't been
+ // provided. This is expected to occur on first-run on platforms that don't
+ // have first-run variations support. This should only be called when there is
+ // no existing field trial controlling the sampling feature, and on the
+ // correct platform and channel.
+ static void CreateFallbackSamplingTrial(base::FeatureList* feature_list);
+
// Determines if this client is eligible to send metrics. If they are, and
// there was user consent, then metrics and crashes would be reported.
static bool IsClientInSample();
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | chrome/browser/metrics/chrome_metrics_services_manager_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698