[Variations] Refactor all state used for study filtering into a container struct.

components/variations/variations_seed_processor.cc

Index: components/variations/variations_seed_processor.cc
diff --git a/components/variations/variations_seed_processor.cc b/components/variations/variations_seed_processor.cc
index c1bb22f5762577aa305ac5f06eafbcf13e33255c..af04452716205aed491093c83489f49ed8ac5880 100644
--- a/components/variations/variations_seed_processor.cc
+++ b/components/variations/variations_seed_processor.cc
@@ -14,6 +14,7 @@
 #include "base/metrics/field_trial.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/utf_string_conversions.h"
+#include "components/variations/client_filterable_state.h"
 #include "components/variations/processed_study.h"
 #include "components/variations/study_filtering.h"
 #include "components/variations/variations_associated_data.h"
@@ -175,26 +176,17 @@ VariationsSeedProcessor::~VariationsSeedProcessor() {
 
 void VariationsSeedProcessor::CreateTrialsFromSeed(
     const VariationsSeed& seed,
-    const std::string& locale,
-    const base::Time& reference_date,
-    const base::Version& version,
-    Study_Channel channel,
-    Study_FormFactor form_factor,
-    const std::string& hardware_class,
-    const std::string& session_consistency_country,
-    const std::string& permanent_consistency_country,
+    const ClientFilterableState& client_state,
     const UIStringOverrideCallback& override_callback,
     const base::FieldTrial::EntropyProvider* low_entropy_provider,
     base::FeatureList* feature_list) {
   std::vector<ProcessedStudy> filtered_studies;
-  FilterAndValidateStudies(seed, locale, reference_date, version, channel,
-                           form_factor, hardware_class,
-                           session_consistency_country,
-                           permanent_consistency_country, &filtered_studies);
+  FilterAndValidateStudies(seed, client_state, &filtered_studies);
 
-  for (size_t i = 0; i < filtered_studies.size(); ++i)
+  for (size_t i = 0; i < filtered_studies.size(); ++i) {

[Alexei Svitkine (slow), 2017/06/07 15:18:08]

Nit: Change to C++11 for loop syntax if you're touching it already.

[Ilya Sherman, 2017/06/07 21:50:08]

Done.

     CreateTrialFromStudy(filtered_studies[i], override_callback,
                          low_entropy_provider, feature_list);
+  }
 }
 
 // static