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

Unified Diff: chrome/browser/net/websocket_experiment/websocket_experiment_runner.cc

Issue 6216004: Feature to disable field trials in old versions of Chromium. Field trials... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 11 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: chrome/browser/net/websocket_experiment/websocket_experiment_runner.cc
===================================================================
--- chrome/browser/net/websocket_experiment/websocket_experiment_runner.cc (revision 71223)
+++ chrome/browser/net/websocket_experiment/websocket_experiment_runner.cc (working copy)
@@ -28,12 +28,13 @@
void WebSocketExperimentRunner::Start() {
DCHECK(!runner.get());
+ // After June 30, 2011 builds, it will always be in defaut group.
scoped_refptr<base::FieldTrial> trial(
- new base::FieldTrial("WebSocketExperiment", 1000));
- trial->AppendGroup("active", 5); // 0.5% in active group.
+ new base::FieldTrial("WebSocketExperiment", 1000, "default", 2011, 6,
+ 30));
+ int active = trial->AppendGroup("active", 5); // 0.5% in active group.
- bool run_experiment =
- (trial->group() != base::FieldTrial::kNotParticipating);
+ bool run_experiment = (trial->group() == active);
#ifndef NDEBUG
const CommandLine& command_line = *CommandLine::ForCurrentProcess();
std::string experiment_host = command_line.GetSwitchValueASCII(

Powered by Google App Engine
This is Rietveld 408576698