Convert base::Bind in stale_host_resolver

Bug: 1007680
Change-Id: I2c42fa8043546f651e4b7cd708f8eae1f66292b8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2283035
Commit-Queue: Quang Minh Tuan Nguyen <nqmtuan@google.com>
Reviewed-by: David Benjamin <davidben@chromium.org>
Reviewed-by: Adam Langley <agl@chromium.org>
Auto-Submit: Quang Minh Tuan Nguyen <nqmtuan@google.com>
Cr-Commit-Position: refs/heads/master@{#787498}

components/cronet/stale_host_resolver.cc

@@ -148,10 +148,10 @@ int StaleHostResolver::RequestImpl::Start(
   if (CacheDataIsUsable()) {
     // |stale_timer_| is deleted when the Request is deleted, so it's safe to
     // use Unretained here.
-    base::Callback<void()> stale_callback =
-        base::Bind(&StaleHostResolver::RequestImpl::OnStaleDelayElapsed,
-                   base::Unretained(this));
-    stale_timer_.Start(FROM_HERE, resolver_->options_.delay, stale_callback);
+    stale_timer_.Start(
+        FROM_HERE, resolver_->options_.delay,
+        base::BindOnce(&StaleHostResolver::RequestImpl::OnStaleDelayElapsed,
+                       base::Unretained(this)));
   } else {
     cache_error_ = net::ERR_DNS_CACHE_MISS;
     cache_request_.reset();

components/cronet/stale_host_resolver_unittest.cc

@@ -290,7 +290,7 @@ class StaleHostResolverTest : public testing::Test {
     // Run until resolve completes or timeout.
     resolve_closure_ = run_loop.QuitWhenIdleClosure();
     base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
-        FROM_HERE, run_loop.QuitWhenIdleClosure(),
+        FROM_HERE, resolve_closure_,
         base::TimeDelta::FromSeconds(kWaitTimeoutSec));
     run_loop.Run();
   }
@@ -364,7 +364,7 @@ class StaleHostResolverTest : public testing::Test {
   bool resolve_complete_;
   int resolve_error_;
 
-  base::Closure resolve_closure_;
+  base::RepeatingClosure resolve_closure_;
 };
 
 // Make sure that test harness can be created and destroyed without crashing.