Chromium Code Reviews| Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_settings_unittest.cc |
| diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings_unittest.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings_unittest.cc |
| index fe091dcecabc32449698e944ec20b4986532acf5..1ec5577b6e72895fa1e64931c06f5c6995938d41 100644 |
| --- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings_unittest.cc |
| +++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings_unittest.cc |
| @@ -194,6 +194,19 @@ TEST(DataReductionProxySettingsStandaloneTest, TestEndToEndSecureProxyCheck) { |
| .SkipSettingsInitialization() |
| .Build(); |
| + // Enabling QUIC should have no effect since secure proxy should not |
| + // use QUIC. If secure proxy check incorrectly uses QUIC, the tests will |
| + // fail because Mock sockets do not speak QUIC. |
| + net::HttpNetworkSession::Params* params = |
|
sclittle
2015/04/02 00:07:00
This memory gets leaked.
sclittle
2015/04/02 00:08:35
Oops, nevermind, it doesn't. Disregard this.
tbansal1
2015/04/02 23:21:20
Done.
|
| + new net::HttpNetworkSession::Params(); |
| + params->use_alternate_protocols = true; |
| + params->enable_quic = true; |
| + params->origin_to_force_quic_on = |
| + net::HostPortPair::FromString( |
| + TestDataReductionProxyParams::DefaultSecureProxyCheckURL()); |
| + |
| + context.set_http_network_session_params(params); |
| + |
| context.set_net_log(drp_test_context->net_log()); |
| net::MockClientSocketFactory mock_socket_factory; |
| context.set_client_socket_factory(&mock_socket_factory); |