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

Unified Diff: net/http/http_network_transaction_unittest.cc

Issue 1720163002: When Alt-Svc header processing is enabled, do not process Alternate-Protocol (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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_network_transaction.cc ('k') | net/quic/quic_network_transaction_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_network_transaction_unittest.cc
diff --git a/net/http/http_network_transaction_unittest.cc b/net/http/http_network_transaction_unittest.cc
index 3f92da8a52a761f847587b06d8f8049ce66ea3b1..bc7f7b37dfbf3d0f33dac274a581e2aad2f2e157 100644
--- a/net/http/http_network_transaction_unittest.cc
+++ b/net/http/http_network_transaction_unittest.cc
@@ -9971,7 +9971,7 @@ TEST_P(HttpNetworkTransactionTest, HonorAlternateProtocolHeader) {
}
TEST_P(HttpNetworkTransactionTest, EmptyAlternateProtocolHeader) {
- session_deps_.parse_alternative_services = true;
+ session_deps_.parse_alternative_services = false;
session_deps_.enable_alternative_service_with_different_host = false;
MockRead data_reads[] = {
@@ -10660,7 +10660,7 @@ TEST_P(HttpNetworkTransactionTest, AlternateProtocolUnsafeBlocked) {
}
TEST_P(HttpNetworkTransactionTest, UseAlternateProtocolForNpnSpdy) {
- session_deps_.parse_alternative_services = true;
+ session_deps_.parse_alternative_services = false;
session_deps_.enable_alternative_service_with_different_host = false;
HttpRequestInfo request;
@@ -10748,7 +10748,7 @@ TEST_P(HttpNetworkTransactionTest, UseAlternateProtocolForNpnSpdy) {
}
TEST_P(HttpNetworkTransactionTest, AlternateProtocolWithSpdyLateBinding) {
- session_deps_.parse_alternative_services = true;
+ session_deps_.parse_alternative_services = false;
session_deps_.enable_alternative_service_with_different_host = false;
HttpRequestInfo request;
@@ -10869,7 +10869,7 @@ TEST_P(HttpNetworkTransactionTest, AlternateProtocolWithSpdyLateBinding) {
}
TEST_P(HttpNetworkTransactionTest, StallAlternateProtocolForNpnSpdy) {
- session_deps_.parse_alternative_services = true;
+ session_deps_.parse_alternative_services = false;
session_deps_.enable_alternative_service_with_different_host = false;
HttpRequestInfo request;
@@ -10998,7 +10998,7 @@ class CapturingProxyResolverFactory : public ProxyResolverFactory {
TEST_P(HttpNetworkTransactionTest,
UseAlternateProtocolForTunneledNpnSpdy) {
- session_deps_.parse_alternative_services = true;
+ session_deps_.parse_alternative_services = false;
session_deps_.enable_alternative_service_with_different_host = false;
ProxyConfig proxy_config;
@@ -11122,7 +11122,7 @@ TEST_P(HttpNetworkTransactionTest,
TEST_P(HttpNetworkTransactionTest,
UseAlternateProtocolForNpnSpdyWithExistingSpdySession) {
- session_deps_.parse_alternative_services = true;
+ session_deps_.parse_alternative_services = false;
session_deps_.enable_alternative_service_with_different_host = false;
HttpRequestInfo request;
@@ -11946,7 +11946,7 @@ class UrlRecordingHttpAuthHandlerMock : public HttpAuthHandlerMock {
// This test ensures that the URL passed into the proxy is upgraded to https
// when doing an Alternate Protocol upgrade.
TEST_P(HttpNetworkTransactionTest, SpdyAlternateProtocolThroughProxy) {
- session_deps_.parse_alternative_services = true;
+ session_deps_.parse_alternative_services = false;
session_deps_.enable_alternative_service_with_different_host = false;
session_deps_.proxy_service =
« no previous file with comments | « net/http/http_network_transaction.cc ('k') | net/quic/quic_network_transaction_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698