diff options
author | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-31 05:14:15 +0000 |
---|---|---|
committer | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-31 05:14:15 +0000 |
commit | 545baea014e3d9b51f6b25b5d6d7f9a429545677 (patch) | |
tree | 8ff5018c5226fce50075cb27e24e4e53bb2dc3ea /chrome/browser/browser_main.cc | |
parent | f32c173f2633b8e5ee74715f6a7a02e286d1de29 (diff) | |
download | chromium_src-545baea014e3d9b51f6b25b5d6d7f9a429545677.zip chromium_src-545baea014e3d9b51f6b25b5d6d7f9a429545677.tar.gz chromium_src-545baea014e3d9b51f6b25b5d6d7f9a429545677.tar.bz2 |
Remove packet split experiment, and add coalescing histogram
The effort to split the packet proved to not work (presumably
showing that the RTO was reasonably set by a SYN packet reception).
The test did show that splitting the first packet had a negative
consequence, and so the question is: How often can we coalesce
2 packets in a request into 1 packet? The most common cause would
be a login, using a POST method to send a body, where the body is
small. This change creates a histogram to show percentages of
requests that can be coalesced.
r=wtc
Review URL: http://codereview.chromium.org/1539003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43180 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_main.cc')
-rw-r--r-- | chrome/browser/browser_main.cc | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/chrome/browser/browser_main.cc b/chrome/browser/browser_main.cc index 017ac15..3913c1d 100644 --- a/chrome/browser/browser_main.cc +++ b/chrome/browser/browser_main.cc @@ -686,20 +686,6 @@ int BrowserMain(const MainFunctionParams& parameters) { // Initialize statistical testing infrastructure for entire browser. FieldTrialList field_trial; - // Set up a field trial to see if splitting the first transmitted packet helps - // with latency. - { - FieldTrial::Probability kDivisor = 100; - FieldTrial* trial = new FieldTrial("PacketSplit", kDivisor); - // For each option (i.e., non-default), we have a fixed probability. - FieldTrial::Probability kProbabilityPerGroup = 10; // 10% probability. - int split = trial->AppendGroup("_first_packet_split", kProbabilityPerGroup); - DCHECK_EQ(split, 0); - int intact = trial->AppendGroup("_first_packet_intact", - FieldTrial::kAllRemainingProbability); - DCHECK_EQ(intact, 1); - } - std::wstring app_name = chrome::kBrowserAppName; std::string thread_name_string = WideToASCII(app_name + L"_BrowserMain"); |