Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(668)

Unified Diff: components/sync/engine/net/http_bridge.cc

Issue 2837863004: Use base::OneShotTimer instead of base::Timer(false, false)
Patch Set: rebase Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/sync/engine/net/http_bridge.cc
diff --git a/components/sync/engine/net/http_bridge.cc b/components/sync/engine/net/http_bridge.cc
index 5c87bed3e6912538fc64c22468e4a9d0cb63cc1a..2bba8effa3ae4f752364897906380c5655e20a05 100644
--- a/components/sync/engine/net/http_bridge.cc
+++ b/components/sync/engine/net/http_bridge.cc
@@ -228,7 +228,7 @@ void HttpBridge::MakeAsynchronousPost() {
// on, and on which the url fetcher lives).
DCHECK(!fetch_state_.http_request_timeout_timer.get());
fetch_state_.http_request_timeout_timer =
- base::MakeUnique<base::Timer>(false, false);
+ base::MakeUnique<base::OneShotTimer>();
fetch_state_.http_request_timeout_timer->Start(
FROM_HERE, base::TimeDelta::FromSeconds(kMaxHttpRequestTimeSeconds),
base::Bind(&HttpBridge::OnURLFetchTimedOut, this));
« no previous file with comments | « components/error_page/renderer/net_error_helper_core.cc ('k') | components/sync/model_impl/attachments/task_queue.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698