Index: net/quic/quic_network_transaction_unittest.cc |
diff --git a/net/quic/quic_network_transaction_unittest.cc b/net/quic/quic_network_transaction_unittest.cc |
index 3d119fbd86355f17be7e963254c8966834bf18dd..3dfa0e3b592cabd3339afd25c4a808fc24f8cb05 100644 |
--- a/net/quic/quic_network_transaction_unittest.cc |
+++ b/net/quic/quic_network_transaction_unittest.cc |
@@ -51,8 +51,6 @@ |
// This is the expected return from a current server advertising QUIC. |
static const char kQuicAlternateProtocolHttpHeader[] = |
"Alternate-Protocol: 80:quic\r\n\r\n"; |
-static const char kQuicAlternateProtocol50pctHttpHeader[] = |
- "Alternate-Protocol: 80:quic,p=.5\r\n\r\n"; |
static const char kQuicAlternateProtocolHttpsHeader[] = |
"Alternate-Protocol: 443:quic\r\n\r\n"; |
@@ -277,13 +275,13 @@ |
MockCryptoClientStream::HandshakeMode handshake_mode) { |
crypto_client_stream_factory_.set_handshake_mode(handshake_mode); |
session_->http_server_properties()->SetAlternateProtocol( |
- HostPortPair::FromURL(request_.url), 80, QUIC, 1); |
+ HostPortPair::FromURL(request_.url), 80, QUIC); |
} |
void ExpectBrokenAlternateProtocolMapping() { |
ASSERT_TRUE(session_->http_server_properties()->HasAlternateProtocol( |
HostPortPair::FromURL(request_.url))); |
- const AlternateProtocolInfo alternate = |
+ const PortAlternateProtocolPair alternate = |
session_->http_server_properties()->GetAlternateProtocol( |
HostPortPair::FromURL(request_.url)); |
EXPECT_EQ(ALTERNATE_PROTOCOL_BROKEN, alternate.protocol); |
@@ -292,7 +290,7 @@ |
void ExpectQuicAlternateProtocolMapping() { |
ASSERT_TRUE(session_->http_server_properties()->HasAlternateProtocol( |
HostPortPair::FromURL(request_.url))); |
- const AlternateProtocolInfo alternate = |
+ const PortAlternateProtocolPair alternate = |
session_->http_server_properties()->GetAlternateProtocol( |
HostPortPair::FromURL(request_.url)); |
EXPECT_EQ(QUIC, alternate.protocol); |
@@ -492,87 +490,6 @@ |
SendRequestAndExpectQuicResponse("hello!"); |
} |
-TEST_P(QuicNetworkTransactionTest, UseAlternateProtocolProbabilityForQuic) { |
- MockRead http_reads[] = { |
- MockRead("HTTP/1.1 200 OK\r\n"), |
- MockRead(kQuicAlternateProtocol50pctHttpHeader), |
- MockRead("hello world"), |
- MockRead(SYNCHRONOUS, ERR_TEST_PEER_CLOSE_AFTER_NEXT_MOCK_READ), |
- MockRead(ASYNC, OK) |
- }; |
- |
- StaticSocketDataProvider http_data(http_reads, arraysize(http_reads), |
- NULL, 0); |
- socket_factory_.AddSocketDataProvider(&http_data); |
- |
- MockQuicData mock_quic_data; |
- mock_quic_data.AddWrite( |
- ConstructRequestHeadersPacket(1, kClientDataStreamId1, true, true, |
- GetRequestHeaders("GET", "http", "/"))); |
- mock_quic_data.AddRead( |
- ConstructResponseHeadersPacket(1, kClientDataStreamId1, false, false, |
- GetResponseHeaders("200 OK"))); |
- mock_quic_data.AddRead( |
- ConstructDataPacket(2, kClientDataStreamId1, false, true, 0, "hello!")); |
- mock_quic_data.AddWrite(ConstructAckPacket(2, 1)); |
- mock_quic_data.AddRead(SYNCHRONOUS, 0); // EOF |
- |
- mock_quic_data.AddDelayedSocketDataToFactory(&socket_factory_, 1); |
- |
- // The non-alternate protocol job needs to hang in order to guarantee that |
- // the alternate-protocol job will "win". |
- AddHangingNonAlternateProtocolSocketData(); |
- |
- params_.alternate_protocol_probability_threshold = .25; |
- CreateSessionWithNextProtos(); |
- |
- SendRequestAndExpectHttpResponse("hello world"); |
- SendRequestAndExpectQuicResponse("hello!"); |
-} |
- |
-TEST_P(QuicNetworkTransactionTest, DontUseAlternateProtocolProbabilityForQuic) { |
- MockRead http_reads[] = { |
- MockRead("HTTP/1.1 200 OK\r\n"), |
- MockRead(kQuicAlternateProtocol50pctHttpHeader), |
- MockRead("hello world"), |
- MockRead(SYNCHRONOUS, ERR_TEST_PEER_CLOSE_AFTER_NEXT_MOCK_READ), |
- MockRead(ASYNC, OK) |
- }; |
- |
- StaticSocketDataProvider http_data(http_reads, arraysize(http_reads), |
- NULL, 0); |
- socket_factory_.AddSocketDataProvider(&http_data); |
- socket_factory_.AddSocketDataProvider(&http_data); |
- |
- params_.alternate_protocol_probability_threshold = .75; |
- CreateSessionWithNextProtos(); |
- |
- SendRequestAndExpectHttpResponse("hello world"); |
- SendRequestAndExpectHttpResponse("hello world"); |
-} |
- |
-TEST_P(QuicNetworkTransactionTest, |
- DontUseAlternateProtocolWithBadProbabilityForQuic) { |
- MockRead http_reads[] = { |
- MockRead("HTTP/1.1 200 OK\r\n"), |
- MockRead("Alternate-Protocol: 443:quic,p=2\r\n\r\n"), |
- MockRead("hello world"), |
- MockRead(SYNCHRONOUS, ERR_TEST_PEER_CLOSE_AFTER_NEXT_MOCK_READ), |
- MockRead(ASYNC, OK) |
- }; |
- |
- StaticSocketDataProvider http_data(http_reads, arraysize(http_reads), |
- NULL, 0); |
- socket_factory_.AddSocketDataProvider(&http_data); |
- socket_factory_.AddSocketDataProvider(&http_data); |
- |
- params_.alternate_protocol_probability_threshold = .75; |
- CreateSessionWithNextProtos(); |
- |
- SendRequestAndExpectHttpResponse("hello world"); |
- SendRequestAndExpectHttpResponse("hello world"); |
-} |
- |
TEST_P(QuicNetworkTransactionTest, UseAlternateProtocolForQuicForHttps) { |
params_.origin_to_force_quic_on = |
HostPortPair::FromString("www.google.com:443"); |