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

Side by Side Diff: components/variations/variations_seed_processor.cc

Issue 1306653004: Expand FeatureList to support FieldTrial association. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 5 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/variations/variations_seed_processor.h" 5 #include "components/variations/variations_seed_processor.h"
6 6
7 #include <map> 7 #include <map>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/command_line.h" 10 #include "base/command_line.h"
11 #include "base/feature_list.h"
11 #include "base/metrics/field_trial.h" 12 #include "base/metrics/field_trial.h"
12 #include "base/strings/utf_string_conversions.h" 13 #include "base/strings/utf_string_conversions.h"
13 #include "components/variations/processed_study.h" 14 #include "components/variations/processed_study.h"
14 #include "components/variations/study_filtering.h" 15 #include "components/variations/study_filtering.h"
15 #include "components/variations/variations_associated_data.h" 16 #include "components/variations/variations_associated_data.h"
16 17
17 namespace variations { 18 namespace variations {
18 19
19 namespace { 20 namespace {
20 21
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 void ApplyUIStringOverrides( 73 void ApplyUIStringOverrides(
73 const Study_Experiment& experiment, 74 const Study_Experiment& experiment,
74 const VariationsSeedProcessor::UIStringOverrideCallback& callback) { 75 const VariationsSeedProcessor::UIStringOverrideCallback& callback) {
75 for (int i = 0; i < experiment.override_ui_string_size(); ++i) { 76 for (int i = 0; i < experiment.override_ui_string_size(); ++i) {
76 const Study_Experiment_OverrideUIString& override = 77 const Study_Experiment_OverrideUIString& override =
77 experiment.override_ui_string(i); 78 experiment.override_ui_string(i);
78 callback.Run(override.name_hash(), base::UTF8ToUTF16(override.value())); 79 callback.Run(override.name_hash(), base::UTF8ToUTF16(override.value()));
79 } 80 }
80 } 81 }
81 82
83 // Forces the specified |experiment| to be enabled in |study|.
84 void ForceExperimentState(
85 const Study& study,
86 const Study_Experiment& experiment,
87 const VariationsSeedProcessor::UIStringOverrideCallback& override_callback,
88 base::FieldTrial* trial) {
89 RegisterExperimentParams(study, experiment);
90 RegisterVariationIds(experiment, study.name());
91 if (study.activation_type() == Study_ActivationType_ACTIVATION_AUTO) {
92 trial->group();
93 // UI Strings can only be overridden from ACTIVATION_AUTO experiments.
94 ApplyUIStringOverrides(experiment, override_callback);
95 }
96 }
97
98 // Registers feature overrides for the chosen experiment in the specified study.
99 void RegisterFeatureOverrides(const ProcessedStudy& processed_study,
100 base::FieldTrial* trial,
101 base::FeatureList* feature_list) {
102 const std::string& group_name = trial->GetGroupNameWithoutActivation();
103 int experiment_index = processed_study.GetExperimentIndexByName(group_name);
104 // The field trial was defined from |study|, so the active experiment's name
105 // must be in the |study|.
106 DCHECK_NE(-1, experiment_index);
107
108 const Study_Experiment& experiment =
109 processed_study.study()->experiment(experiment_index);
110
111 // Process all the features to enable.
112 int feature_count = experiment.feature_association().enable_feature_size();
113 for (int i = 0; i < feature_count; ++i) {
114 feature_list->RegisterFieldTrialOverride(
115 experiment.feature_association().enable_feature(i),
116 base::FeatureList::OVERRIDE_ENABLE_FEATURE, trial);
117 }
118
119 // Process all the features to disable.
120 feature_count = experiment.feature_association().disable_feature_size();
121 for (int i = 0; i < feature_count; ++i) {
122 feature_list->RegisterFieldTrialOverride(
123 experiment.feature_association().disable_feature(i),
124 base::FeatureList::OVERRIDE_DISABLE_FEATURE, trial);
125 }
126 }
127
128 // Checks if |experiment| is associated with a forcing flag or feature and if it
129 // is, returns whether it should be forced enabled based on the |command_line|
130 // or |feature_list| state.
131 bool ShouldForceExperiment(const Study_Experiment& experiment,
132 const base::CommandLine& command_line,
133 const base::FeatureList& feature_list) {
134 if (experiment.feature_association().has_forcing_feature_on()) {
135 return feature_list.IsFeatureOverriddenFromCommandLine(
136 experiment.feature_association().forcing_feature_on(),
137 base::FeatureList::OVERRIDE_ENABLE_FEATURE);
138 }
139 if (experiment.feature_association().has_forcing_feature_off()) {
140 return feature_list.IsFeatureOverriddenFromCommandLine(
141 experiment.feature_association().forcing_feature_off(),
142 base::FeatureList::OVERRIDE_DISABLE_FEATURE);
143 }
144 if (experiment.has_forcing_flag())
145 return command_line.HasSwitch(experiment.forcing_flag());
146 return false;
147 }
148
82 } // namespace 149 } // namespace
83 150
84 VariationsSeedProcessor::VariationsSeedProcessor() { 151 VariationsSeedProcessor::VariationsSeedProcessor() {
85 } 152 }
86 153
87 VariationsSeedProcessor::~VariationsSeedProcessor() { 154 VariationsSeedProcessor::~VariationsSeedProcessor() {
88 } 155 }
89 156
90 void VariationsSeedProcessor::CreateTrialsFromSeed( 157 void VariationsSeedProcessor::CreateTrialsFromSeed(
91 const VariationsSeed& seed, 158 const VariationsSeed& seed,
92 const std::string& locale, 159 const std::string& locale,
93 const base::Time& reference_date, 160 const base::Time& reference_date,
94 const base::Version& version, 161 const base::Version& version,
95 Study_Channel channel, 162 Study_Channel channel,
96 Study_FormFactor form_factor, 163 Study_FormFactor form_factor,
97 const std::string& hardware_class, 164 const std::string& hardware_class,
98 const std::string& session_consistency_country, 165 const std::string& session_consistency_country,
99 const std::string& permanent_consistency_country, 166 const std::string& permanent_consistency_country,
100 const UIStringOverrideCallback& override_callback) { 167 const UIStringOverrideCallback& override_callback,
168 base::FeatureList* feature_list) {
101 std::vector<ProcessedStudy> filtered_studies; 169 std::vector<ProcessedStudy> filtered_studies;
102 FilterAndValidateStudies(seed, locale, reference_date, version, channel, 170 FilterAndValidateStudies(seed, locale, reference_date, version, channel,
103 form_factor, hardware_class, 171 form_factor, hardware_class,
104 session_consistency_country, 172 session_consistency_country,
105 permanent_consistency_country, &filtered_studies); 173 permanent_consistency_country, &filtered_studies);
106 174
107 for (size_t i = 0; i < filtered_studies.size(); ++i) 175 for (size_t i = 0; i < filtered_studies.size(); ++i)
108 CreateTrialFromStudy(filtered_studies[i], override_callback); 176 CreateTrialFromStudy(filtered_studies[i], override_callback, feature_list);
109 } 177 }
110 178
111 void VariationsSeedProcessor::CreateTrialFromStudy( 179 void VariationsSeedProcessor::CreateTrialFromStudy(
112 const ProcessedStudy& processed_study, 180 const ProcessedStudy& processed_study,
113 const UIStringOverrideCallback& override_callback) { 181 const UIStringOverrideCallback& override_callback,
182 base::FeatureList* feature_list) {
114 const Study& study = *processed_study.study(); 183 const Study& study = *processed_study.study();
115 184
116 // Check if any experiments need to be forced due to a command line 185 // Check if any experiments need to be forced due to a command line
117 // flag. Force the first experiment with an existing flag. 186 // flag. Force the first experiment with an existing flag.
118 base::CommandLine* command_line = base::CommandLine::ForCurrentProcess(); 187 base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
119 for (int i = 0; i < study.experiment_size(); ++i) { 188 for (int i = 0; i < study.experiment_size(); ++i) {
120 const Study_Experiment& experiment = study.experiment(i); 189 const Study_Experiment& experiment = study.experiment(i);
121 if (experiment.has_forcing_flag() && 190 if (ShouldForceExperiment(experiment, *command_line, *feature_list)) {
122 command_line->HasSwitch(experiment.forcing_flag())) { 191 base::FieldTrial* trial = base::FieldTrialList::CreateFieldTrial(
123 scoped_refptr<base::FieldTrial> trial( 192 study.name(), experiment.name());
124 base::FieldTrialList::CreateFieldTrial(study.name(), 193 // If |trial| is null, then there might already be a trial forced to a
125 experiment.name()));
126 // If |trial| is NULL, then there might already be a trial forced to a
127 // different group (e.g. via --force-fieldtrials). Break out of the loop, 194 // different group (e.g. via --force-fieldtrials). Break out of the loop,
128 // but don't return, so that variation ids and params for the selected 195 // but don't return, so that variation ids and params for the selected
129 // group will still be picked up. 196 // group will still be picked up.
130 if (!trial.get()) 197 if (!trial)
131 break; 198 break;
132 199
133 RegisterExperimentParams(study, experiment); 200 if (experiment.feature_association().has_forcing_feature_on()) {
134 RegisterVariationIds(experiment, study.name()); 201 feature_list->AssociateReportingFieldTrial(
135 if (study.activation_type() == Study_ActivationType_ACTIVATION_AUTO) { 202 experiment.feature_association().forcing_feature_on(),
136 trial->group(); 203 base::FeatureList::OVERRIDE_ENABLE_FEATURE, trial);
137 // UI Strings can only be overridden from ACTIVATION_AUTO experiments. 204 } else if (experiment.feature_association().has_forcing_feature_off()) {
138 ApplyUIStringOverrides(experiment, override_callback); 205 feature_list->AssociateReportingFieldTrial(
206 experiment.feature_association().forcing_feature_off(),
207 base::FeatureList::OVERRIDE_DISABLE_FEATURE, trial);
139 } 208 }
140 209 ForceExperimentState(study, experiment, override_callback, trial);
141 DVLOG(1) << "Trial " << study.name() << " forced by flag: "
142 << experiment.forcing_flag();
143 return; 210 return;
144 } 211 }
145 } 212 }
146 213
147 uint32 randomization_seed = 0; 214 uint32 randomization_seed = 0;
148 base::FieldTrial::RandomizationType randomization_type = 215 base::FieldTrial::RandomizationType randomization_type =
149 base::FieldTrial::SESSION_RANDOMIZED; 216 base::FieldTrial::SESSION_RANDOMIZED;
150 if (study.has_consistency() && 217 if (study.has_consistency() &&
151 study.consistency() == Study_Consistency_PERMANENT && 218 study.consistency() == Study_Consistency_PERMANENT &&
152 // If all assignments are to a single group, no need to enable one time 219 // If all assignments are to a single group, no need to enable one time
153 // randomization (which is more expensive to compute), since the result 220 // randomization (which is more expensive to compute), since the result
154 // will be the same. 221 // will be the same.
155 !processed_study.all_assignments_to_one_group()) { 222 !processed_study.all_assignments_to_one_group()) {
156 randomization_type = base::FieldTrial::ONE_TIME_RANDOMIZED; 223 randomization_type = base::FieldTrial::ONE_TIME_RANDOMIZED;
157 if (study.has_randomization_seed()) 224 if (study.has_randomization_seed())
158 randomization_seed = study.randomization_seed(); 225 randomization_seed = study.randomization_seed();
159 } 226 }
160 227
161 // The trial is created without specifying an expiration date because the 228 // The trial is created without specifying an expiration date because the
162 // expiration check in field_trial.cc is based on the build date. Instead, 229 // expiration check in field_trial.cc is based on the build date. Instead,
163 // the expiration check using |reference_date| is done explicitly below. 230 // the expiration check using |reference_date| is done explicitly below.
164 scoped_refptr<base::FieldTrial> trial( 231 scoped_refptr<base::FieldTrial> trial(
165 base::FieldTrialList::FactoryGetFieldTrialWithRandomizationSeed( 232 base::FieldTrialList::FactoryGetFieldTrialWithRandomizationSeed(
166 study.name(), processed_study.total_probability(), 233 study.name(), processed_study.total_probability(),
167 study.default_experiment_name(), 234 study.default_experiment_name(),
168 base::FieldTrialList::kNoExpirationYear, 1, 1, randomization_type, 235 base::FieldTrialList::kNoExpirationYear, 1, 1, randomization_type,
169 randomization_seed, NULL)); 236 randomization_seed, NULL));
170 237
171 bool has_overrides = false; 238 bool has_overrides = false;
239 bool enables_or_disables_features = false;
172 for (int i = 0; i < study.experiment_size(); ++i) { 240 for (int i = 0; i < study.experiment_size(); ++i) {
173 const Study_Experiment& experiment = study.experiment(i); 241 const Study_Experiment& experiment = study.experiment(i);
174 RegisterExperimentParams(study, experiment); 242 RegisterExperimentParams(study, experiment);
175 243
176 // Groups with forcing flags have probability 0 and will never be selected. 244 // Groups with forcing flags have probability 0 and will never be selected.
177 // Therefore, there's no need to add them to the field trial. 245 // Therefore, there's no need to add them to the field trial.
178 if (experiment.has_forcing_flag()) 246 if (experiment.has_forcing_flag() ||
247 experiment.feature_association().has_forcing_feature_on() ||
248 experiment.feature_association().has_forcing_feature_off()) {
179 continue; 249 continue;
250 }
180 251
181 if (experiment.name() != study.default_experiment_name()) 252 if (experiment.name() != study.default_experiment_name())
182 trial->AppendGroup(experiment.name(), experiment.probability_weight()); 253 trial->AppendGroup(experiment.name(), experiment.probability_weight());
183 254
184 RegisterVariationIds(experiment, study.name()); 255 RegisterVariationIds(experiment, study.name());
185 256
186 has_overrides = has_overrides || experiment.override_ui_string_size() > 0; 257 has_overrides = has_overrides || experiment.override_ui_string_size() > 0;
258 if (experiment.feature_association().enable_feature_size() != 0 ||
259 experiment.feature_association().disable_feature_size() != 0) {
260 enables_or_disables_features = true;
261 }
187 } 262 }
188 263
189 trial->SetForced(); 264 trial->SetForced();
265
266 if (enables_or_disables_features)
267 RegisterFeatureOverrides(processed_study, trial.get(), feature_list);
268
190 if (processed_study.is_expired()) { 269 if (processed_study.is_expired()) {
191 trial->Disable(); 270 trial->Disable();
192 } else if (study.activation_type() == Study_ActivationType_ACTIVATION_AUTO) { 271 } else if (study.activation_type() == Study_ActivationType_ACTIVATION_AUTO) {
193 const std::string& group_name = trial->group_name(); 272 const std::string& group_name = trial->group_name();
194 273
195 // Don't try to apply overrides if none of the experiments in this study had 274 // Don't try to apply overrides if none of the experiments in this study had
196 // any. 275 // any.
197 if (!has_overrides) 276 if (!has_overrides)
198 return; 277 return;
199 278
200 // UI Strings can only be overridden from ACTIVATION_AUTO experiments. 279 // UI Strings can only be overridden from ACTIVATION_AUTO experiments.
201 int experiment_index = processed_study.GetExperimentIndexByName(group_name); 280 int experiment_index = processed_study.GetExperimentIndexByName(group_name);
202 281
203 // The field trial was defined from |study|, so the active experiment's name 282 // The field trial was defined from |study|, so the active experiment's name
204 // must be in the |study|. 283 // must be in the |study|.
205 DCHECK_NE(-1, experiment_index); 284 DCHECK_NE(-1, experiment_index);
206 285
207 ApplyUIStringOverrides(study.experiment(experiment_index), 286 ApplyUIStringOverrides(study.experiment(experiment_index),
208 override_callback); 287 override_callback);
209 } 288 }
210 } 289 }
211 290
212 } // namespace variations 291 } // namespace variations
OLDNEW
« no previous file with comments | « components/variations/variations_seed_processor.h ('k') | components/variations/variations_seed_processor_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698