Mechanical change of base::Time to base::TimeTicks in the component updater.

The call sites where the time is used to compute time deltas within the
lifetime of the Chrome browser process are affected.

BUG=646619

Review-Url: https://codereview.chromium.org/2336913003
Cr-Commit-Position: refs/heads/master@{#418959}

components/component_updater/component_updater_service.cc

@@ -267,7 +267,8 @@ bool CrxUpdateService::OnDemandUpdateWithCooldown(const std::string& id) {
   // Check if the request is too soon.
   const auto* component_state(GetComponentState(id));
   if (component_state) {
-    base::TimeDelta delta = base::Time::Now() - component_state->last_check;
+    base::TimeDelta delta =
+        base::TimeTicks::Now() - component_state->last_check;
     if (delta < base::TimeDelta::FromSeconds(config_->OnDemandDelay()))
       return false;
   }

components/update_client/action_update_check.cc

@@ -66,7 +66,7 @@ void ActionUpdateCheck::Run(UpdateContext* update_context, Callback callback) {
 
     item->id = GetCrxComponentID(crx_component);
     item->component = crx_component;
-    item->last_check = base::Time::Now();
+    item->last_check = base::TimeTicks::Now();
     item->crx_urls.clear();
     item->crx_diffurls.clear();
     item->previous_version = crx_component.version;

components/update_client/background_downloader_win.cc

@@ -460,7 +460,7 @@ void BackgroundDownloader::DoStartDownload(const GURL& url) {
 void BackgroundDownloader::BeginDownload(const GURL& url) {
   DCHECK(task_runner()->RunsTasksOnCurrentThread());
 
-  download_start_time_ = base::Time::Now();
+  download_start_time_ = base::TimeTicks::Now();
   job_stuck_begin_time_ = download_start_time_;
 
   HRESULT hr = BeginDownloadHelper(url);
@@ -573,7 +573,7 @@ void BackgroundDownloader::EndDownload(HRESULT error) {
 
   DCHECK(!TimerIsRunning());
 
-  const base::Time download_end_time(base::Time::Now());
+  const base::TimeTicks download_end_time(base::TimeTicks::Now());
   const base::TimeDelta download_time =
       download_end_time >= download_start_time_
           ? download_end_time - download_start_time_
@@ -691,7 +691,7 @@ bool BackgroundDownloader::OnStateQueued() {
 bool BackgroundDownloader::OnStateTransferring() {
   // Resets the baseline for detecting a stuck job since the job is transferring
   // data and it is making progress.
-  job_stuck_begin_time_ = base::Time::Now();
+  job_stuck_begin_time_ = base::TimeTicks::Now();
 
   int64_t downloaded_bytes = -1;
   int64_t total_bytes = -1;
@@ -800,7 +800,7 @@ HRESULT BackgroundDownloader::InitializeNewJob(
 bool BackgroundDownloader::IsStuck() {
   const base::TimeDelta job_stuck_timeout(
       base::TimeDelta::FromMinutes(kJobStuckTimeoutMin));
-  return job_stuck_begin_time_ + job_stuck_timeout < base::Time::Now();
+  return job_stuck_begin_time_ + job_stuck_timeout < base::TimeTicks::Now();
 }
 
 HRESULT BackgroundDownloader::CompleteJob() {

components/update_client/background_downloader_win.h

@@ -8,6 +8,8 @@
 #include <windows.h>
 #include <bits.h>
 
+#include <memory>
+
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/strings/string16.h"
@@ -134,10 +136,10 @@ class BackgroundDownloader : public CrxDownloader {
   base::win::ScopedComPtr<IBackgroundCopyJob> job_;
 
   // Contains the time when the download of the current url has started.
-  base::Time download_start_time_;
+  base::TimeTicks download_start_time_;
 
   // Contains the time when the BITS job is last seen making progress.
-  base::Time job_stuck_begin_time_;
+  base::TimeTicks job_stuck_begin_time_;
 
   // Contains the path of the downloaded file if the download was successful.
   base::FilePath response_;

components/update_client/crx_update_item.h

@@ -80,7 +80,7 @@ struct CrxUpdateItem {
   CrxComponent component;
 
   // Time when an update check for this CRX has happened.
-  base::Time last_check;
+  base::TimeTicks last_check;
 
   // Time when the update of this CRX has begun.
   base::TimeTicks update_begin;

components/update_client/update_engine.cc

@@ -128,8 +128,9 @@ void UpdateEngine::UpdateComplete(UpdateContext* update_context, int error) {
   if (throttle_sec >= 0)
     throttle_updates_until_ =
         throttle_sec
-            ? base::Time::Now() + base::TimeDelta::FromSeconds(throttle_sec)
-            : base::Time();
+            ? base::TimeTicks::Now() +
+                  base::TimeDelta::FromSeconds(throttle_sec)
+            : base::TimeTicks();
 
   auto callback = update_context->callback;
 
@@ -143,7 +144,7 @@ bool UpdateEngine::IsThrottled(bool is_foreground) const {
   if (is_foreground || throttle_updates_until_.is_null())
     return false;
 
-  const auto now(base::Time::Now());
+  const auto now(base::TimeTicks::Now());
 
   // Throttle the calls in the interval (t - 1 day, t) to limit the effect of
   // unset clocks or clock drift.

components/update_client/update_engine.h

@@ -25,6 +25,7 @@
 #include "components/update_client/update_client.h"
 
 namespace base {
+class TimeTicks;
 class SequencedTaskRunner;
 class SingleThreadTaskRunner;
 }  // namespace base
@@ -89,7 +90,7 @@ class UpdateEngine {
   // effect of rejecting the update call if the update call occurs before
   // a certain time, which is negotiated with the server as part of the
   // update protocol. See the comments for X-Retry-After header.
-  base::Time throttle_updates_until_;
+  base::TimeTicks throttle_updates_until_;
 
   DISALLOW_COPY_AND_ASSIGN(UpdateEngine);
 };

components/update_client/url_fetcher_downloader.cc

@@ -46,7 +46,7 @@ void UrlFetcherDownloader::DoStartDownload(const GURL& url) {
   VLOG(1) << "Starting background download: " << url.spec();
   url_fetcher_->Start();
 
-  download_start_time_ = base::Time::Now();
+  download_start_time_ = base::TimeTicks::Now();
 
   downloaded_bytes_ = -1;
   total_bytes_ = -1;
@@ -55,7 +55,7 @@ void UrlFetcherDownloader::DoStartDownload(const GURL& url) {
 void UrlFetcherDownloader::OnURLFetchComplete(const net::URLFetcher* source) {
   DCHECK(thread_checker_.CalledOnValidThread());
 
-  const base::Time download_end_time(base::Time::Now());
+  const base::TimeTicks download_end_time(base::TimeTicks::Now());
   const base::TimeDelta download_time =
       download_end_time >= download_start_time_
           ? download_end_time - download_start_time_

components/update_client/url_fetcher_downloader.h

@@ -50,7 +50,7 @@ class UrlFetcherDownloader : public CrxDownloader,
   std::unique_ptr<net::URLFetcher> url_fetcher_;
   net::URLRequestContextGetter* context_getter_;
 
-  base::Time download_start_time_;
+  base::TimeTicks download_start_time_;
 
   int64_t downloaded_bytes_;
   int64_t total_bytes_;