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

Unified Diff: components/variations/variations_seed_processor.cc

Issue 2465003002: Removing Variations support for google_update_experiment_id. (Closed)
Patch Set: Fix typo. 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/variations_seed_processor.cc
diff --git a/components/variations/variations_seed_processor.cc b/components/variations/variations_seed_processor.cc
index b152ddb302c813e886ec27cebf11bafbd0900d44..50c5cdc7dc39219208130d93a1177c97461ff41d 100644
--- a/components/variations/variations_seed_processor.cc
+++ b/components/variations/variations_seed_processor.cc
@@ -54,14 +54,6 @@ void RegisterVariationIds(const Study_Experiment& experiment,
experiment.name(),
variation_id);
}
- if (experiment.has_google_update_experiment_id()) {
- const VariationID variation_id =
- static_cast<VariationID>(experiment.google_update_experiment_id());
- AssociateGoogleVariationIDForce(GOOGLE_UPDATE_SERVICE,
- trial_name,
- experiment.name(),
- variation_id);
- }
if (experiment.has_chrome_sync_experiment_id()) {
const VariationID variation_id =
static_cast<VariationID>(experiment.chrome_sync_experiment_id());
@@ -206,7 +198,6 @@ bool VariationsSeedProcessor::ShouldStudyUseLowEntropy(const Study& study) {
const Study_Experiment& experiment = study.experiment(i);
if (experiment.has_google_web_experiment_id() ||
experiment.has_google_web_trigger_experiment_id() ||
- experiment.has_google_update_experiment_id() ||
experiment.has_chrome_sync_experiment_id()) {
return true;
}
« no previous file with comments | « components/variations/variations_associated_data_unittest.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698