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

Unified Diff: content/renderer/renderer_main.cc

Issue 6883102: Add one-time randomization support for FieldTrial, and the ability to (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Respond to review comments. Created 9 years, 8 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: content/renderer/renderer_main.cc
diff --git a/content/renderer/renderer_main.cc b/content/renderer/renderer_main.cc
index dbb0e34b633aec735d58a0f66ee9f3aced38a2ac..440ef77248e1aafb1be453fc023ff79c170caf8c 100644
--- a/content/renderer/renderer_main.cc
+++ b/content/renderer/renderer_main.cc
@@ -191,8 +191,10 @@ int RendererMain(const MainFunctionParams& parameters) {
statistics.reset(new base::StatisticsRecorder());
}
- // Initialize statistical testing infrastructure.
- base::FieldTrialList field_trial;
+ // Initialize statistical testing infrastructure. We set client_id to the
+ // empty string to disallow the renderer process from creating its own
+ // one-time randomized trials; they should be created in the browser process.
+ base::FieldTrialList field_trial(EmptyString());
// Ensure any field trials in browser are reflected into renderer.
if (parsed_command_line.HasSwitch(switches::kForceFieldTestNameAndValue)) {
std::string persistent = parsed_command_line.GetSwitchValueASCII(

Powered by Google App Engine
This is Rietveld 408576698