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

Unified Diff: components/error_page/renderer/net_error_helper_core.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
« no previous file with comments | « components/drive/file_write_watcher.cc ('k') | components/sync/engine/net/http_bridge.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/error_page/renderer/net_error_helper_core.cc
diff --git a/components/error_page/renderer/net_error_helper_core.cc b/components/error_page/renderer/net_error_helper_core.cc
index 287cd84def6b5900c37cdcf0f84b5affeb397900..6abd0e02e6879eb61d90221500262cc129bfce4e 100644
--- a/components/error_page/renderer/net_error_helper_core.cc
+++ b/components/error_page/renderer/net_error_helper_core.cc
@@ -513,7 +513,7 @@ NetErrorHelperCore::NetErrorHelperCore(Delegate* delegate,
can_show_network_diagnostics_dialog_(false),
auto_reload_enabled_(auto_reload_enabled),
auto_reload_visible_only_(auto_reload_visible_only),
- auto_reload_timer_(new base::Timer(false, false)),
+ auto_reload_timer_(new base::OneShotTimer),
auto_reload_paused_(false),
auto_reload_in_flight_(false),
uncommitted_load_started_(false),
@@ -521,8 +521,7 @@ NetErrorHelperCore::NetErrorHelperCore(Delegate* delegate,
online_(true),
visible_(is_visible),
auto_reload_count_(0),
- navigation_from_button_(NO_BUTTON) {
-}
+ navigation_from_button_(NO_BUTTON) {}
NetErrorHelperCore::~NetErrorHelperCore() {
if (committed_error_page_info_ &&
« no previous file with comments | « components/drive/file_write_watcher.cc ('k') | components/sync/engine/net/http_bridge.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698