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

Unified Diff: chrome/browser/chrome_browser_main.cc

Issue 10005041: Remove SPDY 2.1 support. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove references to deleted flags from chrome/browser Created 8 years, 8 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 | « chrome/browser/about_flags.cc ('k') | chrome/common/chrome_switches.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
diff --git a/chrome/browser/chrome_browser_main.cc b/chrome/browser/chrome_browser_main.cc
index 9b172d1e0116492817afb7fe25394ef013e4592e..23dfae0566bcc27b6a251824b1fb36b3d0b6cc1a 100644
--- a/chrome/browser/chrome_browser_main.cc
+++ b/chrome/browser/chrome_browser_main.cc
@@ -790,19 +790,14 @@ void ChromeBrowserMainParts::SpdyFieldTrial() {
}
if (parsed_command_line().HasSwitch(switches::kEnableSpdy3)) {
net::HttpStreamFactory::EnableNpnSpdy3();
- } else if (parsed_command_line().HasSwitch(
- switches::kEnableSpdyFlowControl)) {
- net::HttpStreamFactory::EnableFlowControl();
} else if (parsed_command_line().HasSwitch(switches::kEnableNpn)) {
net::HttpStreamFactory::EnableNpnSpdy();
} else if (parsed_command_line().HasSwitch(switches::kEnableNpnHttpOnly)) {
net::HttpStreamFactory::EnableNpnHttpOnly();
} else {
#if !defined(OS_CHROMEOS)
- bool is_spdy_trial = false;
const base::FieldTrial::Probability kSpdyDivisor = 100;
base::FieldTrial::Probability npnhttp_probability = 5;
- base::FieldTrial::Probability flow_control_probability = 5;
base::FieldTrial::Probability spdy3_probability = 0;
// After June 30, 2013 builds, it will always be in default group.
@@ -816,28 +811,18 @@ void ChromeBrowserMainParts::SpdyFieldTrial() {
// NPN with only http support, no spdy.
int npn_http_grp = trial->AppendGroup("npn_with_http", npnhttp_probability);
- // NPN with http/1.1, spdy/2, spdy/2.1 and spdy/3 support.
+ // NPN with http/1.1, spdy/2, and spdy/3 support.
int spdy3_grp = trial->AppendGroup("spdy3", spdy3_probability);
- // NPN with http/1.1, spdy/2 and spdy/2.1 support.
- int flow_control_grp = trial->AppendGroup(
- "flow_control", flow_control_probability);
-
int trial_grp = trial->group();
if (trial_grp == npn_spdy_grp) {
- is_spdy_trial = true;
net::HttpStreamFactory::EnableNpnSpdy();
} else if (trial_grp == npn_http_grp) {
- is_spdy_trial = true;
net::HttpStreamFactory::EnableNpnHttpOnly();
} else if (trial_grp == spdy3_grp) {
- is_spdy_trial = true;
net::HttpStreamFactory::EnableNpnSpdy3();
- } else if (trial_grp == flow_control_grp) {
- is_spdy_trial = true;
- net::HttpStreamFactory::EnableFlowControl();
} else {
- CHECK(!is_spdy_trial);
+ NOTREACHED();
}
#else
// Always enable SPDY on Chrome OS
« no previous file with comments | « chrome/browser/about_flags.cc ('k') | chrome/common/chrome_switches.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698