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

Unified Diff: net/http/http_stream_factory_impl.cc

Issue 25956002: [SPDY] Remove references to obsolete SPDY versions SPDY/1 and SPDY/2.1 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix unit test Created 7 years, 2 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 | « net/http/http_stream_factory.cc ('k') | net/socket/next_proto.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_stream_factory_impl.cc
diff --git a/net/http/http_stream_factory_impl.cc b/net/http/http_stream_factory_impl.cc
index 35d94d779f8cefbac1cc7eed79f1c250a44cfca7..1d413f5420a28909af2536484d7ae760a6ad30d3 100644
--- a/net/http/http_stream_factory_impl.cc
+++ b/net/http/http_stream_factory_impl.cc
@@ -207,11 +207,10 @@ PortAlternateProtocolPair HttpStreamFactoryImpl::GetAlternateProtocolRequestFor(
if (alternate.protocol == ALTERNATE_PROTOCOL_BROKEN)
return kNoAlternateProtocol;
- DCHECK_LE(NPN_SPDY_1, alternate.protocol);
- DCHECK_GT(NUM_ALTERNATE_PROTOCOLS, alternate.protocol);
-
- if (alternate.protocol < NPN_SPDY_2)
+ if (!IsAlternateProtocolValid(alternate.protocol)) {
+ NOTREACHED();
return kNoAlternateProtocol;
+ }
// Some shared unix systems may have user home directories (like
// http://foo.com/~mike) which allow users to emit headers. This is a bad
« no previous file with comments | « net/http/http_stream_factory.cc ('k') | net/socket/next_proto.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698