Chromium Code Reviews| Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_config_unittest.cc |
| diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_unittest.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_unittest.cc |
| index 082e0f09a7858b37f7f9c457e2643d12f94f57d5..8614ed4f37a5388094e99bccd154b65039c0bbda 100644 |
| --- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_unittest.cc |
| +++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_unittest.cc |
| @@ -127,13 +127,11 @@ class DataReductionProxyConfigTest : public testing::Test { |
| !config()->restricted_by_carrier_, |
| request_succeeded && (response == "OK")), |
| request_succeeded, 1); |
| - MockDataReductionProxyService* service = |
| - test_context_->mock_data_reduction_proxy_service(); |
| TestResponder responder; |
| responder.response = response; |
| responder.status = |
| net::URLRequestStatus(net::URLRequestStatus::SUCCESS, net::OK); |
| - EXPECT_CALL(*service, SecureProxyCheck(_, _)) |
| + EXPECT_CALL(*(config()), SecureProxyCheck(_, _)) |
|
sclittle
2015/04/02 00:07:00
Could this just be *config() instead of *(config()
tbansal1
2015/04/02 23:21:19
Done.
|
| .Times(1) |
| .WillRepeatedly(testing::WithArgs<1>( |
| testing::Invoke(&responder, &TestResponder::ExecuteCallback))); |
| @@ -161,6 +159,7 @@ class DataReductionProxyConfigTest : public testing::Test { |
| scoped_ptr<DataReductionProxyParams> params) { |
| params->EnableQuic(false); |
| return make_scoped_ptr(new DataReductionProxyConfig( |
| + test_context_-> request_context_getter(), |
|
sclittle
2015/04/02 00:07:00
remove space in the middle
tbansal1
2015/04/02 23:21:20
Done.
|
| test_context_->task_runner(), test_context_->task_runner(), |
| test_context_->net_log(), params.Pass(), test_context_->configurator(), |
| test_context_->event_store())); |