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 b3947fc605cfdd6ff4db088cdc04c5a93b1a5ed8..1813c809ea5f040542ddba3a75334d4e51d2ecf2 100644 |
--- a/net/http/http_network_transaction_unittest.cc |
+++ b/net/http/http_network_transaction_unittest.cc |
@@ -8746,9 +8746,9 @@ TEST_P(HttpNetworkTransactionTest, HonorAlternateProtocolHeader) { |
HostPortPair http_host_port_pair("www.example.org", 80); |
HttpServerProperties& http_server_properties = |
*session->http_server_properties(); |
- AlternativeService alternative_service = |
- http_server_properties.GetAlternativeService(http_host_port_pair); |
- EXPECT_EQ(alternative_service.protocol, UNINITIALIZED_ALTERNATE_PROTOCOL); |
+ AlternativeServiceVector alternative_service_vector = |
+ http_server_properties.GetAlternativeServices(http_host_port_pair); |
+ EXPECT_TRUE(alternative_service_vector.empty()); |
EXPECT_EQ(OK, callback.WaitForResult()); |
@@ -8763,11 +8763,12 @@ TEST_P(HttpNetworkTransactionTest, HonorAlternateProtocolHeader) { |
ASSERT_EQ(OK, ReadTransaction(trans.get(), &response_data)); |
EXPECT_EQ("hello world", response_data); |
- alternative_service = |
- http_server_properties.GetAlternativeService(http_host_port_pair); |
- EXPECT_EQ(443, alternative_service.port); |
+ alternative_service_vector = |
+ http_server_properties.GetAlternativeServices(http_host_port_pair); |
+ ASSERT_EQ(1u, alternative_service_vector.size()); |
+ EXPECT_EQ(443, alternative_service_vector[0].port); |
EXPECT_EQ(AlternateProtocolFromNextProto(GetParam()), |
- alternative_service.protocol); |
+ alternative_service_vector[0].protocol); |
} |
TEST_P(HttpNetworkTransactionTest, EmptyAlternateProtocolHeader) { |
@@ -8801,9 +8802,10 @@ TEST_P(HttpNetworkTransactionTest, EmptyAlternateProtocolHeader) { |
http_server_properties.SetAlternativeService(http_host_port_pair, |
alternative_service, 1.0); |
- alternative_service = |
- http_server_properties.GetAlternativeService(http_host_port_pair); |
- EXPECT_EQ(alternative_service.protocol, QUIC); |
+ AlternativeServiceVector alternative_service_vector = |
+ http_server_properties.GetAlternativeServices(http_host_port_pair); |
+ ASSERT_EQ(1u, alternative_service_vector.size()); |
+ EXPECT_EQ(QUIC, alternative_service_vector[0].protocol); |
scoped_ptr<HttpTransaction> trans( |
new HttpNetworkTransaction(DEFAULT_PRIORITY, session.get())); |
@@ -8824,9 +8826,9 @@ TEST_P(HttpNetworkTransactionTest, EmptyAlternateProtocolHeader) { |
ASSERT_EQ(OK, ReadTransaction(trans.get(), &response_data)); |
EXPECT_EQ("hello world", response_data); |
- alternative_service = |
- http_server_properties.GetAlternativeService(http_host_port_pair); |
- EXPECT_EQ(alternative_service.protocol, UNINITIALIZED_ALTERNATE_PROTOCOL); |
+ alternative_service_vector = |
+ http_server_properties.GetAlternativeServices(http_host_port_pair); |
+ EXPECT_TRUE(alternative_service_vector.empty()); |
} |
TEST_P(HttpNetworkTransactionTest, |
@@ -8859,9 +8861,9 @@ TEST_P(HttpNetworkTransactionTest, |
const HostPortPair host_port_pair = HostPortPair::FromURL(request.url); |
// Port must be < 1024, or the header will be ignored (since initial port was |
// port 80 (another restricted port). |
- AlternativeService alternative_service( |
+ const AlternativeService alternative_service( |
AlternateProtocolFromNextProto(GetParam()), "www.example.org", |
- 666); /* port is ignored by MockConnect anyway */ |
+ 666); // Port is ignored by MockConnect anyway. |
http_server_properties->SetAlternativeService(host_port_pair, |
alternative_service, 1.0); |
@@ -8882,11 +8884,12 @@ TEST_P(HttpNetworkTransactionTest, |
ASSERT_EQ(OK, ReadTransaction(trans.get(), &response_data)); |
EXPECT_EQ("hello world", response_data); |
- alternative_service = |
- http_server_properties->GetAlternativeService(host_port_pair); |
- EXPECT_NE(UNINITIALIZED_ALTERNATE_PROTOCOL, alternative_service.protocol); |
- EXPECT_TRUE( |
- http_server_properties->IsAlternativeServiceBroken(alternative_service)); |
+ const AlternativeServiceVector alternative_service_vector = |
+ http_server_properties->GetAlternativeServices(host_port_pair); |
+ ASSERT_EQ(1u, alternative_service_vector.size()); |
+ EXPECT_EQ(alternative_service, alternative_service_vector[0]); |
+ EXPECT_TRUE(http_server_properties->IsAlternativeServiceBroken( |
+ alternative_service_vector[0])); |
} |
TEST_P(HttpNetworkTransactionTest, |