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

Unified Diff: components/variations/field_trial_config/field_trial_util.h

Issue 2465953002: Move Field Trial Utils from chrome/common/variations to components/variations/field_trial_util (Closed)
Patch Set: Moved to components/variations/field_trial_util and chrome switches 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/variations/field_trial_config/field_trial_util.h
diff --git a/chrome/common/variations/variations_util.h b/components/variations/field_trial_config/field_trial_util.h
similarity index 75%
rename from chrome/common/variations/variations_util.h
rename to components/variations/field_trial_config/field_trial_util.h
index fe84dc8de49060480de758f13e8d87563417cac5..e9487c4ef67e27c7b5ee57ffb7ad7e83e2bbc321 100644
--- a/chrome/common/variations/variations_util.h
+++ b/components/variations/field_trial_config/field_trial_util.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CHROME_COMMON_VARIATIONS_VARIATIONS_UTIL_H_
-#define CHROME_COMMON_VARIATIONS_VARIATIONS_UTIL_H_
+#ifndef COMPONENTS_VARIATIONS_FIELD_TRIAL_CONFIG_FIELD_TRIAL_UTIL_H_
+#define COMPONENTS_VARIATIONS_FIELD_TRIAL_CONFIG_FIELD_TRIAL_UTIL_H_
#include <string>
@@ -11,14 +11,14 @@ namespace base {
class FeatureList;
}
-namespace chrome_variations {
+namespace variations {
struct FieldTrialTestingConfig;
// Provides a mechanism to associate multiple set of params to multiple groups
// with a formatted string specified from commandline. See
-// kForceFieldTrialParams in chrome/common/chrome_switches.cc for more details
-// on the formatting.
+// kForceFieldTrialParams in components/variations/variations_switches.cc for
+// more details on the formatting.
bool AssociateParamsFromString(const std::string& variations_string);
// Provides a mechanism to associate multiple set of params and features to
@@ -29,10 +29,10 @@ void AssociateParamsFromFieldTrialConfig(const FieldTrialTestingConfig& config,
base::FeatureList* feature_list);
// Associates params and features to FieldTrial groups and forces the selection
-// of groups specified in testing/variations/fieldtrial_testing_config_*.json.
+// of groups specified in testing/variations/fieldtrial_testing_config.json.
// Registers features associated with default field trials with |feature_list|.
void AssociateDefaultFieldTrialConfig(base::FeatureList* feature_list);
-} // namespace chrome_variations
+} // namespace variations
-#endif // CHROME_COMMON_VARIATIONS_VARIATIONS_UTIL_H_
+#endif // COMPONENTS_VARIATIONS_FIELD_TRIAL_CONFIG_FIELD_TRIAL_UTIL_H_

Powered by Google App Engine
This is Rietveld 408576698