Index: chrome/browser/prerender/prerender_field_trial.cc |
diff --git a/chrome/browser/prerender/prerender_field_trial.cc b/chrome/browser/prerender/prerender_field_trial.cc |
index a8e8991df723ba631ba9a38eafab797e3f0d6f3a..be1e1ae5cd05a25feb4733a758e7de78b84f1afe 100644 |
--- a/chrome/browser/prerender/prerender_field_trial.cc |
+++ b/chrome/browser/prerender/prerender_field_trial.cc |
@@ -48,32 +48,37 @@ void SetupPrefetchFieldTrial() { |
void SetupPrerenderFieldTrial() { |
base::FieldTrial::Probability divisor = 1000; |
- base::FieldTrial::Probability exp1_probability = 166; |
- base::FieldTrial::Probability exp1_5min_ttl_probability = 83; |
- base::FieldTrial::Probability control1_probability = 166; |
- base::FieldTrial::Probability no_use1_probability = 83; |
- |
- base::FieldTrial::Probability exp2_probability = 167; |
- base::FieldTrial::Probability exp2_5min_ttl_probability = 84; |
- base::FieldTrial::Probability control2_probability = 167; |
- base::FieldTrial::Probability no_use2_probability = 84; |
+ base::FieldTrial::Probability exp1_probability = 150; |
dominich
2012/07/19 16:04:38
I would very much like to remove half of these now
gavinp
2012/07/23 18:35:21
I like that plan, but I figured a separate CL for
|
+ base::FieldTrial::Probability exp1_5min_ttl_probability = 67; |
+ base::FieldTrial::Probability control1_probability = 150; |
+ base::FieldTrial::Probability no_use1_probability = 67; |
+ base::FieldTrial::Probability one_prerender1_probability = 66; |
dominich
2012/07/19 16:04:38
We should flip this around. have exp[12] be single
gavinp
2012/07/23 18:35:21
Done.
|
+ |
+ base::FieldTrial::Probability exp2_probability = 150; |
+ base::FieldTrial::Probability exp2_5min_ttl_probability = 67; |
+ base::FieldTrial::Probability control2_probability = 150; |
+ base::FieldTrial::Probability no_use2_probability = 67; |
+ base::FieldTrial::Probability one_prerender2_probability = 66; |
chrome::VersionInfo::Channel channel = chrome::VersionInfo::GetChannel(); |
if (channel == chrome::VersionInfo::CHANNEL_STABLE || |
channel == chrome::VersionInfo::CHANNEL_BETA) { |
- exp1_probability = 490; |
+ exp1_probability = 0; |
exp1_5min_ttl_probability = 5; |
control1_probability = 5; |
no_use1_probability = 0; |
- exp2_probability = 490; |
+ one_prerender1_probability = 490; |
+ exp2_probability = 0; |
exp2_5min_ttl_probability = 5; |
control2_probability = 5; |
no_use2_probability = 0; |
+ one_prerender2_probability = 490; |
} |
CHECK_EQ(divisor, exp1_probability + exp1_5min_ttl_probability + |
- control1_probability + no_use1_probability + exp2_probability + |
+ control1_probability + no_use1_probability + |
+ one_prerender1_probability + exp2_probability + |
exp2_5min_ttl_probability + control2_probability + |
- no_use2_probability); |
+ no_use2_probability + one_prerender2_probability); |
int experiment_1_group = -1; |
scoped_refptr<base::FieldTrial> trial( |
base::FieldTrialList::FactoryGetFieldTrial( |
@@ -89,6 +94,9 @@ void SetupPrerenderFieldTrial() { |
const int no_use_1_group = |
trial->AppendGroup("ContentPrefetchPrerenderNoUse1", |
no_use1_probability); |
+ const int one_prerender_1_group = |
+ trial->AppendGroup("ContentPrefetchPrerenderOne", |
+ one_prerender1_probability); |
const int experiment_2_group = |
trial->AppendGroup("ContentPrefetchPrerender2", |
exp2_probability); |
@@ -101,6 +109,9 @@ void SetupPrerenderFieldTrial() { |
const int no_use_2_group = |
trial->AppendGroup("ContentPrefetchPrerenderNoUse2", |
no_use2_probability); |
+ const int one_prerender_2_group = |
+ trial->AppendGroup("ContentPrefetchPrerenderOne", |
+ one_prerender2_probability); |
const int trial_group = trial->group(); |
if (trial_group == experiment_1_group || |
trial_group == experiment_2_group) { |
@@ -118,6 +129,10 @@ void SetupPrerenderFieldTrial() { |
trial_group == no_use_2_group) { |
PrerenderManager::SetMode( |
PrerenderManager::PRERENDER_MODE_EXPERIMENT_NO_USE_GROUP); |
+ } else if (trial_group == one_prerender_1_group || |
+ trial_group == one_prerender_2_group) { |
+ PrerenderManager::SetMode( |
+ PrerenderManager::PRERENDER_MODE_EXPERIMENT_ONE_PRERENDER_GROUP); |
} else { |
NOTREACHED(); |
} |