Activate prepopulated fieldtrials

chrome/common/variations/variations_util.cc

Index: chrome/common/variations/variations_util.cc
diff --git a/chrome/common/variations/variations_util.cc b/chrome/common/variations/variations_util.cc
index 35b5bfd0d8053e5547fe4773808d6a738c6d913e..dfb65112be3e96887dddc26163b8c42b96447e26 100644
--- a/chrome/common/variations/variations_util.cc
+++ b/chrome/common/variations/variations_util.cc
@@ -7,9 +7,11 @@
 #include <string>
 #include <vector>
 
+#include "base/metrics/field_trial.h"
 #include "base/strings/string_split.h"
 #include "chrome/common/child_process_logging.h"
 #include "chrome/common/crash_keys.h"
+#include "chrome/common/variations/fieldtrial_testing_config.h"
 #include "components/variations/active_field_trials.h"
 #include "components/variations/variations_associated_data.h"
 #include "net/base/escape.h"
@@ -63,4 +65,26 @@ bool AssociateParamsFromString(const std::string& varations_string) {
   return true;
 }
 
+bool AssociateParamsFromFieldTrialConfig(
+    const FieldTrialTestingConfig& config) {
+  for (size_t i = 0; i < config.groups_size; ++i) {
+    const FieldTrialTestingGroup& group = config.groups[i];
+    if (group.params_size != 0) {
+      std::map<std::string, std::string> params;
+      for (size_t j = 0; j < group.params_size; ++j) {
+        const FieldTrialGroupParams& param = group.params[j];
+        params[param.key] = param.value;
+      }
+      variations::AssociateVariationParams(group.study, group.group_name,
+                                           params);
+    }
+    base::FieldTrialList::CreateFieldTrial(group.study, group.group_name);
+  }
+  return true;

[brettw, 2015/07/15 17:56:51]

This function never fails, but the code carefully propagates the values and
CHECKS later on. I'd just remove all of that and have it return void.

+}
+
+bool AssociateDefaultFieldTrialConfig() {
+  return AssociateParamsFromFieldTrialConfig(kFieldTrialConfig);
+}
+
 }  // namespace chrome_variations