Index: net/proxy/dhcp_proxy_script_fetcher_win_unittest.cc |
diff --git a/net/proxy/dhcp_proxy_script_fetcher_win_unittest.cc b/net/proxy/dhcp_proxy_script_fetcher_win_unittest.cc |
index 9c2dc2489aaa12323f5f37e108da58b80443c854..b3148ddb0182f5e1d769e9dc6fe671ba6e324630 100644 |
--- a/net/proxy/dhcp_proxy_script_fetcher_win_unittest.cc |
+++ b/net/proxy/dhcp_proxy_script_fetcher_win_unittest.cc |
@@ -49,8 +49,6 @@ class RealFetchTester { |
: context_((new TestURLRequestContext())), |
fetcher_(new DhcpProxyScriptFetcherWin(context_.get())), |
finished_(false), |
- ALLOW_THIS_IN_INITIALIZER_LIST( |
- completion_callback_(this, &RealFetchTester::OnCompletion)), |
on_completion_is_error_(false) { |
// Make sure the test ends. |
timeout_.Start(FROM_HERE, |
@@ -58,7 +56,9 @@ class RealFetchTester { |
} |
void RunTest() { |
- int result = fetcher_->Fetch(&pac_text_, &completion_callback_); |
+ int result = fetcher_->Fetch( |
+ &pac_text_, |
+ base::Bind(&RealFetchTester::OnCompletion, base::Unretained(this))); |
if (result != ERR_IO_PENDING) |
finished_ = true; |
} |
@@ -116,7 +116,6 @@ class RealFetchTester { |
scoped_ptr<DhcpProxyScriptFetcherWin> fetcher_; |
bool finished_; |
string16 pac_text_; |
- OldCompletionCallbackImpl<RealFetchTester> completion_callback_; |
base::OneShotTimer<RealFetchTester> timeout_; |
base::OneShotTimer<RealFetchTester> cancel_timer_; |
bool on_completion_is_error_; |
@@ -216,13 +215,12 @@ class DummyDhcpProxyScriptAdapterFetcher |
did_finish_(false), |
result_(OK), |
pac_script_(L"bingo"), |
- fetch_delay_ms_(1), |
- client_callback_(NULL) { |
+ fetch_delay_ms_(1) { |
} |
void Fetch(const std::string& adapter_name, |
- OldCompletionCallback* callback) OVERRIDE { |
- client_callback_ = callback; |
+ const CompletionCallback& callback) OVERRIDE { |
+ callback_ = callback; |
timer_.Start(FROM_HERE, base::TimeDelta::FromMilliseconds(fetch_delay_ms_), |
this, &DummyDhcpProxyScriptAdapterFetcher::OnTimer); |
} |
@@ -244,7 +242,7 @@ class DummyDhcpProxyScriptAdapterFetcher |
} |
void OnTimer() { |
- client_callback_->Run(result_); |
+ callback_.Run(result_); |
} |
void Configure( |
@@ -260,7 +258,7 @@ class DummyDhcpProxyScriptAdapterFetcher |
int result_; |
string16 pac_script_; |
int fetch_delay_ms_; |
- OldCompletionCallback* client_callback_; |
+ CompletionCallback callback_; |
base::OneShotTimer<DummyDhcpProxyScriptAdapterFetcher> timer_; |
}; |
@@ -372,13 +370,13 @@ class FetcherClient { |
public: |
FetcherClient() |
: finished_(false), |
- result_(ERR_UNEXPECTED), |
- ALLOW_THIS_IN_INITIALIZER_LIST( |
- completion_callback_(this, &FetcherClient::OnCompletion)) { |
+ result_(ERR_UNEXPECTED) { |
} |
void RunTest() { |
- int result = fetcher_.Fetch(&pac_text_, &completion_callback_); |
+ int result = fetcher_.Fetch( |
+ &pac_text_, |
+ base::Bind(&FetcherClient::OnCompletion, base::Unretained(this))); |
ASSERT_EQ(ERR_IO_PENDING, result); |
} |
@@ -413,7 +411,6 @@ public: |
bool finished_; |
int result_; |
string16 pac_text_; |
- OldCompletionCallbackImpl<FetcherClient> completion_callback_; |
}; |
// We separate out each test's logic so that we can easily implement |