Replace base::Timer with behavior-specific timers in extensions/

This CL replaces base::Timer with its subclasses.
As base::Timer changes its behavior subject to its construction time
flags, that makes hard to see the actual timer behavior, especially
it's unclear when a timer is injected from other components.
Also, that OnceCallback support of base::Timer is hard to implement on
the dynamically determined invocation pattern.

Bug: 850247
Change-Id: I2a7a85a39ba25da01cec2b151d2911f7dba8029a
Reviewed-on: https://chromium-review.googlesource.com/1128691Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#573524}

chrome/browser/extensions/api/easy_unlock_private/easy_unlock_private_api.h

@@ -18,7 +18,7 @@
 // Implementations for chrome.easyUnlockPrivate API functions.
 
 namespace base {
-class Timer;
+class OneShotTimer;
 }
 
 namespace content {
@@ -139,7 +139,7 @@ class EasyUnlockPrivateFindSetupConnectionFunction
       connection_finder_;
 
   // Used for timing out when waiting for the connection finder to return.
-  std::unique_ptr<base::Timer> timer_;
+  std::unique_ptr<base::OneShotTimer> timer_;
 
   DISALLOW_COPY_AND_ASSIGN(EasyUnlockPrivateFindSetupConnectionFunction);
 };

chrome/browser/extensions/api/tab_capture/offscreen_tab.cc

@@ -94,7 +94,6 @@ OffscreenTab::OffscreenTab(OffscreenTabsOwner* owner)
                       owner->extension_web_contents()->GetBrowserContext()),
                   base::BindOnce(&OffscreenTab::DieIfOriginalProfileDestroyed,
                                  base::Unretained(this)))),
-      capture_poll_timer_(false, false),
       content_capture_was_detected_(false),
       navigation_policy_(
           std::make_unique<media_router::DefaultNavigationPolicy>()) {

chrome/browser/extensions/api/tab_capture/offscreen_tab.h

@@ -219,7 +219,7 @@ class OffscreenTab : protected content::WebContentsDelegate,
   // TODO(miu): Add a method to WebContentsObserver to report capturer count
   // changes and get rid of this polling-based approach.
   // http://crbug.com/540965
-  base::Timer capture_poll_timer_;
+  base::OneShotTimer capture_poll_timer_;
 
   // This is false until after the Start() method is called, and capture of the
   // |offscreen_tab_web_contents_| is first detected.

extensions/browser/updater/request_queue.h

@@ -111,7 +111,7 @@ class RequestQueue {
 
   // Timer to schedule calls to StartNextRequest, if the first pending request
   // hasn't passed its release time yet.
-  base::Timer timer_;
+  base::OneShotTimer timer_;
 };
 
 // Iterator class that wraps a base::circular_deque<> iterator, only giving

extensions/browser/updater/request_queue_impl.h

@@ -22,9 +22,7 @@ RequestQueue<T>::RequestQueue(
     const net::BackoffEntry::Policy* const backoff_policy,
     const base::Closure& start_request_callback)
     : backoff_policy_(backoff_policy),
-      start_request_callback_(start_request_callback),
-      timer_(false, false) {
-}
+      start_request_callback_(start_request_callback) {}
 
 template <typename T>
 RequestQueue<T>::~RequestQueue() {

extensions/renderer/api/display_source/wifi_display/wifi_display_session.cc

@@ -140,9 +140,10 @@ void WiFiDisplaySession::SendRTSPData(const std::string& message) {
 }
 
 unsigned WiFiDisplaySession::CreateTimer(int seconds) {
-  std::unique_ptr<base::Timer> timer(new base::Timer(true, true));
-  auto insert_ret = timers_.insert(std::pair<int, std::unique_ptr<base::Timer>>(
-      ++timer_id_, std::move(timer)));
+  std::unique_ptr<base::RepeatingTimer> timer(new base::RepeatingTimer());
+  auto insert_ret =
+      timers_.insert(std::pair<int, std::unique_ptr<base::RepeatingTimer>>(
+          ++timer_id_, std::move(timer)));
   DCHECK(insert_ret.second);
   insert_ret.first->second->Start(FROM_HERE,
                base::TimeDelta::FromSeconds(seconds),

extensions/renderer/api/display_source/wifi_display/wifi_display_session.h

@@ -14,7 +14,7 @@
 #include "third_party/wds/src/libwds/public/source.h"
 
 namespace base {
-class Timer;
+class RepeatingTimer;
 }  // namespace base
 
 namespace extensions {
@@ -77,7 +77,7 @@ class WiFiDisplaySession : public DisplaySourceSession,
   mojom::WiFiDisplaySessionServicePtr service_;
   mojo::Binding<WiFiDisplaySessionServiceClient> binding_;
   std::string local_ip_address_;
-  std::map<int, std::unique_ptr<base::Timer>> timers_;
+  std::map<int, std::unique_ptr<base::RepeatingTimer>> timers_;
 
   DisplaySourceSessionParams params_;
   CompletionCallback start_completion_callback_;