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

Unified Diff: net/http/http_network_transaction_unittest.cc

Issue 2082403002: Change GroupName test to https. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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 | no next file » | 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 f1a3a2d733107727ba4edef0d645204e14334c2a..e3fd3511e4cd539a405fea1d69164c9bdaaecb9d 100644
--- a/net/http/http_network_transaction_unittest.cc
+++ b/net/http/http_network_transaction_unittest.cc
@@ -9096,10 +9096,10 @@ std::unique_ptr<HttpNetworkSession> SetupSessionForGroupNameTests(
HttpServerProperties* http_server_properties =
session->http_server_properties();
AlternativeService alternative_service(
- AlternateProtocolFromNextProto(next_proto), "", 443);
+ AlternateProtocolFromNextProto(next_proto), "", 444);
base::Time expiration = base::Time::Now() + base::TimeDelta::FromDays(1);
http_server_properties->SetAlternativeService(
- url::SchemeHostPort("http", "host.with.alternate", 80),
+ url::SchemeHostPort("https", "host.with.alternate", 443),
alternative_service, expiration);
return session;
@@ -9153,7 +9153,7 @@ TEST_P(HttpNetworkTransactionTest, GroupNameForDirectConnections) {
},
{
"", // unused
- "http://host.with.alternate/direct",
+ "https://host.with.alternate/direct",
"ssl/host.with.alternate:443",
true,
},
@@ -9208,7 +9208,7 @@ TEST_P(HttpNetworkTransactionTest, GroupNameForHTTPProxyConnections) {
{
"http_proxy",
- "http://host.with.alternate/direct",
+ "https://host.with.alternate/direct",
"ssl/host.with.alternate:443",
true,
},
@@ -9285,7 +9285,7 @@ TEST_P(HttpNetworkTransactionTest, GroupNameForSOCKSConnections) {
{
"socks4://socks_proxy:1080",
- "http://host.with.alternate/direct",
+ "https://host.with.alternate/direct",
"socks4/ssl/host.with.alternate:443",
true,
},
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698