diff options
| author | ziadh@chromium.org <ziadh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-31 02:30:14 +0000 |
|---|---|---|
| committer | ziadh@chromium.org <ziadh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-31 02:30:14 +0000 |
| commit | 990ace19f23253d0d7c66ad4095797ca5d481545 (patch) | |
| tree | d275a94d21495ee9af14a5e595e18c04c70e2cb9 | |
| parent | a324a45fada4feccc9175b1ca37e3cf8be99148c (diff) | |
| download | chromium_src-990ace19f23253d0d7c66ad4095797ca5d481545.zip chromium_src-990ace19f23253d0d7c66ad4095797ca5d481545.tar.gz chromium_src-990ace19f23253d0d7c66ad4095797ca5d481545.tar.bz2 | |
Revert 54411 - Reduce magnitude of non-default group probabilities for 2 A/B tests
I reduced the non-default group probabilities for "ConnCountImpact" and "IdleSktToImpact" by an order of magnitude from [20%, 25%] to [2%, 2.5%] respectively.
BUG=50802
r=jar
Review URL: http://codereview.chromium.org/3056037
TBR=ziadh@chromium.org
Review URL: http://codereview.chromium.org/2806099
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54423 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/browser_main.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/chrome/browser/browser_main.cc b/chrome/browser/browser_main.cc index f0976b9..277cac1 100644 --- a/chrome/browser/browser_main.cc +++ b/chrome/browser/browser_main.cc @@ -175,8 +175,8 @@ void BrowserMainParts::EarlyInitialization() { // on browsing. Too large a value might cause us to run into SYN flood detection // mechanisms. void BrowserMainParts::ConnectionFieldTrial() { - const FieldTrial::Probability kConnectDivisor = 1000; - const FieldTrial::Probability kConnectProbability = 20; // 2% probability + const FieldTrial::Probability kConnectDivisor = 100; + const FieldTrial::Probability kConnectProbability = 20; // 20% probability scoped_refptr<FieldTrial> connect_trial = new FieldTrial("ConnCountImpact", kConnectDivisor); @@ -219,8 +219,8 @@ void BrowserMainParts::ConnectionFieldTrial() { // result in more ERR_CONNECT_RESETs, requiring one RTT to receive the RST // packet and possibly another RTT to re-establish the connection. void BrowserMainParts::SocketTimeoutFieldTrial() { - const FieldTrial::Probability kIdleSocketTimeoutDivisor = 1000; - // 2.5% probability + const FieldTrial::Probability kIdleSocketTimeoutDivisor = 100; + // 25% probability const FieldTrial::Probability kSocketTimeoutProbability = 25; scoped_refptr<FieldTrial> socket_timeout_trial = |