Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_interceptor_unittest.cc |
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_interceptor_unittest.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_interceptor_unittest.cc |
index 7205ac422ccd41f8a75a2dd11bea2d2c418ff3eb..fbacf8fdca225809b946f26c1990156a5fca906a 100644 |
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_interceptor_unittest.cc |
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_interceptor_unittest.cc |
@@ -94,11 +94,11 @@ class CountingURLRequestInterceptor : public net::URLRequestInterceptor { |
class TestURLRequestContextWithDataReductionProxy |
: public net::TestURLRequestContext { |
public: |
- TestURLRequestContextWithDataReductionProxy( |
- TestDataReductionProxyConfig* config, net::NetworkDelegate* delegate) |
+ TestURLRequestContextWithDataReductionProxy(const net::ProxyServer& origin, |
+ net::NetworkDelegate* delegate) |
: net::TestURLRequestContext(true) { |
- std::string proxy = config->test_params()->origin().ToURI(); |
- context_storage_.set_proxy_service(net::ProxyService::CreateFixed(proxy)); |
+ context_storage_.set_proxy_service( |
+ net::ProxyService::CreateFixed(origin.ToURI())); |
set_network_delegate(delegate); |
} |
@@ -113,9 +113,9 @@ class DataReductionProxyInterceptorTest : public testing::Test { |
.WithParamsFlags(DataReductionProxyParams::kAllowed) |
.WithParamsDefinitions(TestDataReductionProxyParams::HAS_EVERYTHING) |
.Build(); |
- default_context_.reset( |
- new TestURLRequestContextWithDataReductionProxy( |
- test_context_->config(), &default_network_delegate_)); |
+ default_context_.reset(new TestURLRequestContextWithDataReductionProxy( |
+ test_context_->config()->test_params()->proxies_for_http(false).front(), |
+ &default_network_delegate_)); |
default_context_->set_network_delegate(&default_network_delegate_); |
default_context_->set_net_log(test_context_->net_log()); |
} |
@@ -204,10 +204,13 @@ class DataReductionProxyInterceptorWithServerTest : public testing::Test { |
.Build(); |
std::string spec; |
base::TrimString(proxy_.GetURL("/").spec(), "/", &spec); |
- test_context_->config()->test_params()->set_origin( |
- net::ProxyServer::FromURI(spec, net::ProxyServer::SCHEME_HTTP)); |
- std::string proxy_name = |
- test_context_->config()->test_params()->origin().ToURI(); |
+ net::ProxyServer origin = |
+ net::ProxyServer::FromURI(spec, net::ProxyServer::SCHEME_HTTP); |
+ std::vector<net::ProxyServer> proxies_for_http; |
+ proxies_for_http.push_back(origin); |
+ test_context_->config()->test_params()->SetProxiesForHttp( |
+ proxies_for_http, std::vector<net::ProxyServer>()); |
+ std::string proxy_name = origin.ToURI(); |
proxy_service_.reset( |
net::ProxyService::CreateFixedFromPacResult( |
"PROXY " + proxy_name + "; DIRECT")); |
@@ -320,6 +323,10 @@ class DataReductionProxyInterceptorEndToEndTest : public testing::Test { |
return drp_test_context_->config(); |
} |
+ net::ProxyServer origin() const { |
+ return config()->test_params()->proxies_for_http(false).front(); |
+ } |
+ |
private: |
base::MessageLoopForIO message_loop_; |
net::TestDelegate delegate_; |
@@ -348,7 +355,7 @@ TEST_F(DataReductionProxyInterceptorEndToEndTest, ResponseWithoutRetry) { |
EXPECT_EQ(net::URLRequestStatus::SUCCESS, request->status().status()); |
EXPECT_EQ(200, request->GetResponseCode()); |
EXPECT_EQ(kBody, delegate().data_received()); |
- EXPECT_EQ(config()->test_params()->origin().host_port_pair().ToString(), |
+ EXPECT_EQ(origin().host_port_pair().ToString(), |
request->proxy_server().ToString()); |
} |
@@ -382,7 +389,7 @@ TEST_F(DataReductionProxyInterceptorEndToEndTest, RedirectWithoutRetry) { |
EXPECT_EQ(net::URLRequestStatus::SUCCESS, request->status().status()); |
EXPECT_EQ(200, request->GetResponseCode()); |
EXPECT_EQ(kBody, delegate().data_received()); |
- EXPECT_EQ(config()->test_params()->origin().host_port_pair().ToString(), |
+ EXPECT_EQ(origin().host_port_pair().ToString(), |
request->proxy_server().ToString()); |
// The redirect should have been processed and followed normally. |
EXPECT_EQ(1, delegate().received_redirect_count()); |