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

Unified Diff: components/metrics/file_metrics_provider.h

Issue 2524363003: Support experiment configurations with/without stability metrics. (Closed)
Patch Set: Created 4 years, 1 month 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: components/metrics/file_metrics_provider.h
diff --git a/components/metrics/file_metrics_provider.h b/components/metrics/file_metrics_provider.h
index ba2e9f9571a0a9bc0734a9a676804d7015588bd1..0d378aee215250ba08aa9626c64f8a6cfcdbabe1 100644
--- a/components/metrics/file_metrics_provider.h
+++ b/components/metrics/file_metrics_provider.h
@@ -101,6 +101,12 @@ class FileMetricsProvider : public MetricsProvider {
static void RegisterPrefs(PrefRegistrySimple* prefs,
const base::StringPiece prefs_key);
+ // Allow stability metrics support (those found in a file for a previous run
+ // of the browser) to be enabled/disabled. By default it is enabled.
+ static void SetStabilityMetricsEnabled(bool enabled) {
+ stability_metrics_enabled_ = enabled;
Alexei Svitkine (slow) 2016/11/24 21:27:58 Nit: Either use hacker_style for the function name
bcwhite 2016/11/25 18:51:58 Done.
+ }
+
private:
friend class FileMetricsProviderTest;
@@ -196,6 +202,9 @@ class FileMetricsProvider : public MetricsProvider {
base::ThreadChecker thread_checker_;
base::WeakPtrFactory<FileMetricsProvider> weak_factory_;
+ // Flag indicating if stability metrics are enabled or disabled.
+ static bool stability_metrics_enabled_;
+
DISALLOW_COPY_AND_ASSIGN(FileMetricsProvider);
};

Powered by Google App Engine
This is Rietveld 408576698