summaryrefslogtreecommitdiffstats
path: root/chrome/browser/browser_main.cc
diff options
context:
space:
mode:
authorjar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-09-16 04:12:15 +0000
committerjar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-09-16 04:12:15 +0000
commit780f849517c85d5596bf5b6b4843da0ab3087eca (patch)
tree7af5f609d0f8d758e90bb579914115d12c88ee60 /chrome/browser/browser_main.cc
parent7a4c685b6ea64503334571fdafff5b2be684b9cc (diff)
downloadchromium_src-780f849517c85d5596bf5b6b4843da0ab3087eca.zip
chromium_src-780f849517c85d5596bf5b6b4843da0ab3087eca.tar.gz
chromium_src-780f849517c85d5596bf5b6b4843da0ab3087eca.tar.bz2
Reduce fieldtrials percentages for push to stable M7
In one case, completely removed the field trial. BUG=55634 r=mbelshe,willchan Review URL: http://codereview.chromium.org/3449003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59614 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_main.cc')
-rw-r--r--chrome/browser/browser_main.cc41
1 files changed, 16 insertions, 25 deletions
diff --git a/chrome/browser/browser_main.cc b/chrome/browser/browser_main.cc
index be78ce6..cd3cc4f 100644
--- a/chrome/browser/browser_main.cc
+++ b/chrome/browser/browser_main.cc
@@ -246,8 +246,8 @@ void BrowserMainParts::ConnectionFieldTrial() {
// packet and possibly another RTT to re-establish the connection.
void BrowserMainParts::SocketTimeoutFieldTrial() {
const FieldTrial::Probability kIdleSocketTimeoutDivisor = 100;
- // 25% probability
- const FieldTrial::Probability kSocketTimeoutProbability = 25;
+ // 1% probability for all experimental settings.
+ const FieldTrial::Probability kSocketTimeoutProbability = 1;
scoped_refptr<FieldTrial> socket_timeout_trial =
new FieldTrial("IdleSktToImpact", kIdleSocketTimeoutDivisor);
@@ -255,19 +255,14 @@ void BrowserMainParts::SocketTimeoutFieldTrial() {
const int socket_timeout_5 =
socket_timeout_trial->AppendGroup("idle_timeout_5",
kSocketTimeoutProbability);
+ const int socket_timeout_10 =
+ socket_timeout_trial->AppendGroup("idle_timeout_10",
+ kSocketTimeoutProbability);
const int socket_timeout_20 =
socket_timeout_trial->AppendGroup("idle_timeout_20",
kSocketTimeoutProbability);
const int socket_timeout_60 =
socket_timeout_trial->AppendGroup("idle_timeout_60",
- kSocketTimeoutProbability);
- // This (10 seconds) is the current default value. Declaring it at the end
- // allows for assigning the remainder of the probability space to it; which
- // will make it to modify this value (by changing |kSocketTimeoutProbability|)
- // down the road if we see the need to, while the remaining groups are
- // are assigned an equal share of the probability space.
- const int socket_timeout_10 =
- socket_timeout_trial->AppendGroup("idle_timeout_10",
FieldTrial::kAllRemainingProbability);
const int idle_to_trial_group = socket_timeout_trial->group();
@@ -339,18 +334,14 @@ void BrowserMainParts::SpdyFieldTrial() {
net::HttpNetworkLayer::EnableSpdy(spdy_mode);
} else {
const FieldTrial::Probability kSpdyDivisor = 1000;
- // To enable 100% npn_with_spdy, set npnhttp_probability = 0 and set
- // npnspdy_probability = FieldTrial::kAllRemainingProbability.
- FieldTrial::Probability npnhttp_probability = 50;
- FieldTrial::Probability npnspdy_probability = 950;
+ FieldTrial::Probability npnhttp_probability = 10; // 1% to preclude SPDY.
scoped_refptr<FieldTrial> trial =
new FieldTrial("SpdyImpact", kSpdyDivisor);
// npn with only http support, no spdy.
- int npn_http_grp =
- trial->AppendGroup("npn_with_http", npnhttp_probability);
+ int npn_http_grp = trial->AppendGroup("npn_with_http", npnhttp_probability);
// npn with spdy support.
- int npn_spdy_grp =
- trial->AppendGroup("npn_with_spdy", npnspdy_probability);
+ int npn_spdy_grp = trial->AppendGroup("npn_with_spdy",
+ FieldTrial::kAllRemainingProbability);
int trial_grp = trial->group();
if (trial_grp == npn_http_grp) {
is_spdy_trial = true;
@@ -365,7 +356,7 @@ void BrowserMainParts::SpdyFieldTrial() {
}
// If neither --enable-content-prefetch or --disable-content-prefetch
-// is set, users will be in an A/B test for prefetching.
+// is set, users will not be in an A/B test for prefetching.
void BrowserMainParts::PrefetchFieldTrial() {
if (parsed_command_line().HasSwitch(switches::kEnableContentPrefetch))
ResourceDispatcherHost::set_is_prefetch_enabled(true);
@@ -400,14 +391,14 @@ void BrowserMainParts::ConnectBackupJobsFieldTrial() {
} else {
const FieldTrial::Probability kConnectBackupJobsDivisor = 100;
// 50% probability.
- const FieldTrial::Probability kConnectBackupJobsProbability = 50;
- scoped_refptr<FieldTrial> trial =
- new FieldTrial("ConnnectBackupJobs", kConnectBackupJobsDivisor);
+ const FieldTrial::Probability kConnectBackupJobsProbability = 1; // 1%.
+ scoped_refptr<FieldTrial> trial = new FieldTrial("ConnnectBackupJobs",
+ kConnectBackupJobsDivisor);
+ trial->AppendGroup("ConnectBackupJobsDisabled",
+ kConnectBackupJobsProbability);
const int connect_backup_jobs_enabled =
trial->AppendGroup("ConnectBackupJobsEnabled",
- kConnectBackupJobsProbability);
- trial->AppendGroup("ConnectBackupJobsDisabled",
- FieldTrial::kAllRemainingProbability);
+ FieldTrial::kAllRemainingProbability);
const int trial_group = trial->group();
net::internal::ClientSocketPoolBaseHelper::set_connect_backup_jobs_enabled(
trial_group == connect_backup_jobs_enabled);