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

Unified Diff: chrome/browser/chrome_browser_field_trials.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
« no previous file with comments | « no previous file | chrome/browser/chrome_browser_field_trials.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chrome_browser_field_trials.h
diff --git a/chrome/browser/chrome_browser_field_trials.h b/chrome/browser/chrome_browser_field_trials.h
index f8c6fea4f4ffecb7122f937e9de03d2465cb814d..ff8c51670ad95099cfb309810267a6b29cd2e178 100644
--- a/chrome/browser/chrome_browser_field_trials.h
+++ b/chrome/browser/chrome_browser_field_trials.h
@@ -9,6 +9,7 @@
#include "base/macros.h"
namespace base {
+class FeatureList;
class Time;
}
@@ -19,6 +20,15 @@ class ChromeBrowserFieldTrials {
void SetupFieldTrials();
+ // Create field trials that will control feature list features. This should be
+ // called during the same timing window as
+ // FeatureList::AssociateReportingFieldTrial. |has_seed| indicates that the
+ // variations service used a seed to create field trials. This can be used to
+ // prevent associating a field trial with a feature that you expect to be
+ // controlled by the variations seed.
+ void SetupFeatureControllingFieldTrials(bool has_seed,
+ base::FeatureList* feature_list);
+
private:
// Instantiates dynamic trials by querying their state, to ensure they get
// reported as used.
« no previous file with comments | « no previous file | chrome/browser/chrome_browser_field_trials.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698