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

Unified Diff: chrome/browser/chrome_browser_main.cc

Issue 16115002: Delete long-expired GlobalSdch field trial. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: address comment Created 7 years, 7 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 | « no previous file | chrome/renderer/page_load_histograms.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chrome_browser_main.cc
===================================================================
--- chrome/browser/chrome_browser_main.cc (revision 202348)
+++ chrome/browser/chrome_browser_main.cc (working copy)
@@ -1402,25 +1402,7 @@
if (!sdch_enabled)
net::SdchManager::EnableSdchSupport(false);
}
- if (sdch_enabled) {
- // Perform A/B test to measure global impact of SDCH support.
- // Set up a field trial to see what disabling SDCH does to latency of page
- // layout globally.
- base::FieldTrial::Probability kSDCH_DIVISOR = 1000;
- base::FieldTrial::Probability kSDCH_DISABLE_PROBABILITY = 1; // 0.1% prob.
- // After March 31, 2012 builds, it will always be in default group.
- int sdch_enabled_group = -1;
- scoped_refptr<base::FieldTrial> sdch_trial(
- base::FieldTrialList::FactoryGetFieldTrial(
- "GlobalSdch", kSDCH_DIVISOR, "global_enable_sdch", 2012, 3, 31,
- &sdch_enabled_group));
- sdch_trial->AppendGroup("global_disable_sdch",
- kSDCH_DISABLE_PROBABILITY);
- if (sdch_enabled_group != sdch_trial->group())
- net::SdchManager::EnableSdchSupport(false);
- }
-
if (parsed_command_line().HasSwitch(switches::kEnableWatchdog))
InstallJankometer(parsed_command_line());
« no previous file with comments | « no previous file | chrome/renderer/page_load_histograms.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698