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

Unified Diff: net/quic/quic_stream_factory_test.cc

Issue 1878143005: SHP 4: Change AlternativeServiceMap to use SchemeHostPort as the key. No change to Pref data. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@SHP_3
Patch Set: fix a comment Created 4 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
Index: net/quic/quic_stream_factory_test.cc
diff --git a/net/quic/quic_stream_factory_test.cc b/net/quic/quic_stream_factory_test.cc
index 50d9e44855ebf0c704dffe5482a7481f460a522e..c642093f7bef5a6cb826751106886c4f81b084fd 100644
--- a/net/quic/quic_stream_factory_test.cc
+++ b/net/quic/quic_stream_factory_test.cc
@@ -2700,8 +2700,11 @@ TEST_P(QuicStreamFactoryTest, RacingConnections) {
alternative_service_info_vector.push_back(
AlternativeServiceInfo(alternative_service1, expiration));
+ url::SchemeHostPort server("https", kDefaultServerHostName,
+ kDefaultServerPort);
+
http_server_properties_.SetAlternativeServices(
- host_port_pair_, alternative_service_info_vector);
+ server, alternative_service_info_vector);
crypto_client_stream_factory_.set_handshake_mode(
MockCryptoClientStream::ZERO_RTT);
@@ -3800,18 +3803,20 @@ TEST_P(QuicStreamFactoryTest, MaybeInitialize) {
base::Time expiration = base::Time::Now() + base::TimeDelta::FromDays(1);
alternative_service_info_vector.push_back(
AlternativeServiceInfo(alternative_service1, expiration));
-
+ url::SchemeHostPort server("https", kDefaultServerHostName,
+ kDefaultServerPort);
http_server_properties_.SetAlternativeServices(
- host_port_pair_, alternative_service_info_vector);
+ server, alternative_service_info_vector);
HostPortPair host_port_pair2(kServer2HostName, kDefaultServerPort);
+ url::SchemeHostPort server2("https", kServer2HostName, kDefaultServerPort);
const AlternativeService alternative_service2(QUIC, host_port_pair2.host(),
host_port_pair2.port());
AlternativeServiceInfoVector alternative_service_info_vector2;
alternative_service_info_vector2.push_back(
AlternativeServiceInfo(alternative_service2, expiration));
http_server_properties_.SetAlternativeServices(
- host_port_pair2, alternative_service_info_vector2);
+ server2, alternative_service_info_vector2);
http_server_properties_.SetMaxServerConfigsStoredInProperties(
kMaxQuicServersToPersist);

Powered by Google App Engine
This is Rietveld 408576698