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

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

Issue 6317004: 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
« no previous file with comments | « chrome/browser/net/predictor_api.cc ('k') | chrome/renderer/renderer_main.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/websocket_experiment/websocket_experiment_runner.cc
===================================================================
--- chrome/browser/net/websocket_experiment/websocket_experiment_runner.cc (revision 71810)
+++ 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 default 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(
« no previous file with comments | « chrome/browser/net/predictor_api.cc ('k') | chrome/renderer/renderer_main.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698