Change WorkerThread::time_origin_ to use base::TimeTicks

Using double to represent time is error prone as the units are unclear.

Change-Id: Id3a6ce23e684643f424dcbc84e2cca708faeda3c
Reviewed-on: https://chromium-review.googlesource.com/1115904Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Adithya Srinivasan <adithyas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570879}

third_party/blink/renderer/core/timing/worker_performance.cc

@@ -41,7 +41,7 @@
 namespace blink {
 
 WorkerPerformance::WorkerPerformance(WorkerGlobalScope* context)
-    : Performance(TimeTicksFromSeconds(context->TimeOrigin()),
+    : Performance(context->TimeOrigin(),
                   context->GetTaskRunner(TaskType::kPerformanceTimeline)),
       execution_context_(context) {}
 

third_party/blink/renderer/core/workers/dedicated_worker_global_scope.cc

@@ -52,7 +52,7 @@ namespace blink {
 DedicatedWorkerGlobalScope::DedicatedWorkerGlobalScope(
     std::unique_ptr<GlobalScopeCreationParams> creation_params,
     DedicatedWorkerThread* thread,
-    double time_origin)
+    base::TimeTicks time_origin)
     : WorkerGlobalScope(std::move(creation_params), thread, time_origin) {}
 
 DedicatedWorkerGlobalScope::~DedicatedWorkerGlobalScope() = default;

third_party/blink/renderer/core/workers/dedicated_worker_global_scope.h

@@ -50,7 +50,7 @@ class CORE_EXPORT DedicatedWorkerGlobalScope final : public WorkerGlobalScope {
  public:
   DedicatedWorkerGlobalScope(std::unique_ptr<GlobalScopeCreationParams>,
                              DedicatedWorkerThread*,
-                             double time_origin);
+                             base::TimeTicks time_origin);
   ~DedicatedWorkerGlobalScope() override;
 
   bool IsDedicatedWorkerGlobalScope() const override { return true; }

third_party/blink/renderer/core/workers/shared_worker_global_scope.cc

@@ -52,7 +52,7 @@ SharedWorkerGlobalScope::SharedWorkerGlobalScope(
     const String& name,
     std::unique_ptr<GlobalScopeCreationParams> creation_params,
     SharedWorkerThread* thread,
-    double time_origin)
+    base::TimeTicks time_origin)
     : WorkerGlobalScope(std::move(creation_params), thread, time_origin),
       name_(name) {}
 

third_party/blink/renderer/core/workers/shared_worker_global_scope.h

@@ -49,7 +49,7 @@ class SharedWorkerGlobalScope final : public WorkerGlobalScope {
   SharedWorkerGlobalScope(const String& name,
                           std::unique_ptr<GlobalScopeCreationParams>,
                           SharedWorkerThread*,
-                          double time_origin);
+                          base::TimeTicks time_origin);
   ~SharedWorkerGlobalScope() override;
 
   bool IsSharedWorkerGlobalScope() const override { return true; }

third_party/blink/renderer/core/workers/worker_global_scope.cc

@@ -329,7 +329,7 @@ void WorkerGlobalScope::EvaluateClassicScript(
 WorkerGlobalScope::WorkerGlobalScope(
     std::unique_ptr<GlobalScopeCreationParams> creation_params,
     WorkerThread* thread,
-    double time_origin)
+    base::TimeTicks time_origin)
     : WorkerOrWorkletGlobalScope(thread->GetIsolate(),
                                  creation_params->worker_clients,
                                  thread->GetWorkerReportingProxy()),

third_party/blink/renderer/core/workers/worker_global_scope.h

@@ -141,7 +141,7 @@ class CORE_EXPORT WorkerGlobalScope
       const FetchClientSettingsObjectSnapshot& outside_settings_object,
       network::mojom::FetchCredentialsMode) = 0;
 
-  double TimeOrigin() const { return time_origin_; }
+  base::TimeTicks TimeOrigin() const { return time_origin_; }
   WorkerSettings* GetWorkerSettings() const { return worker_settings_.get(); }
 
   void Trace(blink::Visitor*) override;
@@ -160,7 +160,7 @@ class CORE_EXPORT WorkerGlobalScope
  protected:
   WorkerGlobalScope(std::unique_ptr<GlobalScopeCreationParams>,
                     WorkerThread*,
-                    double time_origin);
+                    base::TimeTicks time_origin);
   void ApplyContentSecurityPolicyFromHeaders(
       const ContentSecurityPolicyResponseHeaders&);
 
@@ -217,7 +217,7 @@ class CORE_EXPORT WorkerGlobalScope
 
   DOMTimerCoordinator timers_;
 
-  const double time_origin_;
+  const base::TimeTicks time_origin_;
 
   HeapHashMap<int, Member<ErrorEvent>> pending_error_events_;
   int last_pending_error_event_id_ = 0;

third_party/blink/renderer/core/workers/worker_thread.cc

@@ -353,7 +353,7 @@ void WorkerThread::ChildThreadTerminatedOnWorkerThread(WorkerThread* child) {
 
 WorkerThread::WorkerThread(ThreadableLoadingContext* loading_context,
                            WorkerReportingProxy& worker_reporting_proxy)
-    : time_origin_(CurrentTimeTicksInSeconds()),
+    : time_origin_(CurrentTimeTicks()),
       worker_thread_id_(GetNextWorkerThreadId()),
       forcible_termination_delay_(kForcibleTerminationDelay),
       devtools_worker_token_(base::UnguessableToken::Create()),

third_party/blink/renderer/core/workers/worker_thread.h

@@ -227,7 +227,7 @@ class CORE_EXPORT WorkerThread : public WebThread::TaskObserver {
 
   // Official moment of creation of worker: when the worker thread is created.
   // (https://w3c.github.io/hr-time/#time-origin)
-  const double time_origin_;
+  const base::TimeTicks time_origin_;
 
  private:
   friend class WorkerThreadTest;

third_party/blink/renderer/core/workers/worker_thread_test_helper.h

@@ -63,7 +63,7 @@ class FakeWorkerGlobalScope : public WorkerGlobalScope {
       WorkerThread* thread)
       : WorkerGlobalScope(std::move(creation_params),
                           thread,
-                          CurrentTimeTicksInSeconds()) {}
+                          CurrentTimeTicks()) {}
 
   ~FakeWorkerGlobalScope() override = default;
 

third_party/blink/renderer/modules/serviceworkers/service_worker_global_scope.cc

@@ -75,7 +75,7 @@ namespace blink {
 ServiceWorkerGlobalScope* ServiceWorkerGlobalScope::Create(
     ServiceWorkerThread* thread,
     std::unique_ptr<GlobalScopeCreationParams> creation_params,
-    double time_origin) {
+    base::TimeTicks time_origin) {
   // If the script is being loaded via script streaming, the script is not yet
   // loaded.
   if (thread->GetInstalledScriptsManager() &&
@@ -94,7 +94,7 @@ ServiceWorkerGlobalScope* ServiceWorkerGlobalScope::Create(
 ServiceWorkerGlobalScope::ServiceWorkerGlobalScope(
     std::unique_ptr<GlobalScopeCreationParams> creation_params,
     ServiceWorkerThread* thread,
-    double time_origin)
+    base::TimeTicks time_origin)
     : WorkerGlobalScope(std::move(creation_params), thread, time_origin) {}
 
 ServiceWorkerGlobalScope::~ServiceWorkerGlobalScope() = default;

third_party/blink/renderer/modules/serviceworkers/service_worker_global_scope.h

@@ -60,7 +60,7 @@ class MODULES_EXPORT ServiceWorkerGlobalScope final : public WorkerGlobalScope {
   static ServiceWorkerGlobalScope* Create(
       ServiceWorkerThread*,
       std::unique_ptr<GlobalScopeCreationParams>,
-      double time_origin);
+      base::TimeTicks time_origin);
 
   ~ServiceWorkerGlobalScope() override;
   bool IsServiceWorkerGlobalScope() const override { return true; }
@@ -134,7 +134,7 @@ class MODULES_EXPORT ServiceWorkerGlobalScope final : public WorkerGlobalScope {
  private:
   ServiceWorkerGlobalScope(std::unique_ptr<GlobalScopeCreationParams>,
                            ServiceWorkerThread*,
-                           double time_origin);
+                           base::TimeTicks time_origin);
   void importScripts(const Vector<String>& urls, ExceptionState&) override;
   SingleCachedMetadataHandler* CreateWorkerScriptCachedMetadataHandler(
       const KURL& script_url,