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

Unified Diff: components/variations/variations_seed_simulator.h

Issue 1984003002: Use low entropy for studies that send experiment IDs to Google properties. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: components/variations/variations_seed_simulator.h
diff --git a/components/variations/variations_seed_simulator.h b/components/variations/variations_seed_simulator.h
index dafaf098afe5f9bde01222f3826232a41070abc3..c18e6c3210a10a31db24d10143e2fc60e685b60c 100644
--- a/components/variations/variations_seed_simulator.h
+++ b/components/variations/variations_seed_simulator.h
@@ -44,9 +44,14 @@ class VariationsSeedSimulator {
~Result();
};
- // Creates the simulator with the given entropy |provider|.
- explicit VariationsSeedSimulator(
- const base::FieldTrial::EntropyProvider& provider);
+ // Creates the simulator with the given default and low entropy providers. The
+ // |low_entropy_provider| will be used for studies that should only use a low
+ // entropy source. This is defined by
+ // VariationsSeedProcessor::ShouldStudyUseLowEntropy, in
+ // variations_seed_processor.h.
+ VariationsSeedSimulator(
+ const base::FieldTrial::EntropyProvider& default_entropy_provider,
+ const base::FieldTrial::EntropyProvider& low_entropy_provider);
virtual ~VariationsSeedSimulator();
// Computes differences between the current process' field trial state and
@@ -97,7 +102,8 @@ class VariationsSeedSimulator {
ChangeType SessionStudyGroupChanged(const ProcessedStudy& filtered_study,
const std::string& selected_group);
- const base::FieldTrial::EntropyProvider& entropy_provider_;
+ const base::FieldTrial::EntropyProvider& default_entropy_provider_;
+ const base::FieldTrial::EntropyProvider& low_entropy_provider_;
DISALLOW_COPY_AND_ASSIGN(VariationsSeedSimulator);
};
« no previous file with comments | « components/variations/variations_seed_processor_unittest.cc ('k') | components/variations/variations_seed_simulator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698