ref: Use Date.now() instead of new Date().getTime() (#7492)

This is slightly shorter and should actually be slightly more performant.

Author: mydea

packages/browser/test/integration/polyfills/raf.js

@@ -17,7 +17,7 @@
 
   if (!window.requestAnimationFrame)
     window.requestAnimationFrame = function (callback, _element) {
-      var currTime = new Date().getTime();
+      var currTime = Date.now();
       var timeToCall = Math.max(0, 16 - (currTime - lastTime));
       var id = window.setTimeout(function () {
         callback(currTime + timeToCall);

packages/integrations/test/offline.test.ts

@@ -194,7 +194,7 @@ function prepopulateEvents(count: number = 1): void {
   for (let i = 0; i < count; i++) {
     events.push({
       message: 'There was an error!',
-      timestamp: new Date().getTime(),
+      timestamp: Date.now(),
     });
   }
 }

packages/replay/src/coreHandlers/handleHistory.ts

@@ -9,7 +9,7 @@ interface HistoryHandlerData {
 function handleHistory(handlerData: HistoryHandlerData): ReplayPerformanceEntry {
   const { from, to } = handlerData;
 
-  const now = new Date().getTime() / 1000;
+  const now = Date.now() / 1000;
 
   return {
     type: 'navigation.push',

packages/replay/src/replay.ts

@@ -78,7 +78,7 @@ export class ReplayContainer implements ReplayContainerInterface {
   /**
    * Timestamp of the last user activity. This lives across sessions.
    */
-  private _lastActivity: number = new Date().getTime();
+  private _lastActivity: number = Date.now();
 
   /**
    * Is the integration currently active?
@@ -108,7 +108,7 @@ export class ReplayContainer implements ReplayContainerInterface {
     traceIds: new Set(),
     urls: [],
     earliestEvent: null,
-    initialTimestamp: new Date().getTime(),
+    initialTimestamp: Date.now(),
     initialUrl: '',
   };
 
@@ -442,7 +442,7 @@ export class ReplayContainer implements ReplayContainerInterface {
     this._clearContext();
 
     this._context.initialUrl = url;
-    this._context.initialTimestamp = new Date().getTime();
+    this._context.initialTimestamp = Date.now();
     this._context.urls.push(url);
   }
 
@@ -634,14 +634,14 @@ export class ReplayContainer implements ReplayContainerInterface {
   /**
    * Update user activity (across session lifespans)
    */
-  private _updateUserActivity(_lastActivity: number = new Date().getTime()): void {
+  private _updateUserActivity(_lastActivity: number = Date.now()): void {
     this._lastActivity = _lastActivity;
   }
 
   /**
    * Updates the session's last activity timestamp
    */
-  private _updateSessionActivity(_lastActivity: number = new Date().getTime()): void {
+  private _updateSessionActivity(_lastActivity: number = Date.now()): void {
     if (this.session) {
       this.session.lastActivity = _lastActivity;
       this._maybeSaveSession();
@@ -768,7 +768,7 @@ export class ReplayContainer implements ReplayContainerInterface {
         eventContext,
         session: this.session,
         options: this.getOptions(),
-        timestamp: new Date().getTime(),
+        timestamp: Date.now(),
       });
     } catch (err) {
       this._handleException(err);

packages/replay/src/session/Session.ts

@@ -7,7 +7,7 @@ import { isSampled } from '../util/isSampled';
  * Get a session with defaults & applied sampling.
  */
 export function makeSession(session: Partial<Session> & { sampled: Sampled }): Session {
-  const now = new Date().getTime();
+  const now = Date.now();
   const id = session.id || uuid4();
   // Note that this means we cannot set a started/lastActivity of `0`, but this should not be relevant outside of tests.
   const started = session.started || now;

packages/replay/src/util/addEvent.ts

@@ -31,7 +31,7 @@ export async function addEvent(
   // page has been left open and idle for a long period of time and user
   // comes back to trigger a new session. The performance entries rely on
   // `performance.timeOrigin`, which is when the page first opened.
-  if (timestampInMs + replay.timeouts.sessionIdle < new Date().getTime()) {
+  if (timestampInMs + replay.timeouts.sessionIdle < Date.now()) {
     return null;
   }
 

packages/replay/src/util/addMemoryEntry.ts

@@ -33,7 +33,7 @@ function createMemoryEntry(memoryEntry: MemoryInfo): ReplayMemoryEntry {
   const { jsHeapSizeLimit, totalJSHeapSize, usedJSHeapSize } = memoryEntry;
   // we don't want to use `getAbsoluteTime` because it adds the event time to the
   // time origin, so we get the current timestamp instead
-  const time = new Date().getTime() / 1000;
+  const time = Date.now() / 1000;
   return {
     type: 'memory',
     name: 'memory',

packages/replay/src/util/createBreadcrumb.ts

@@ -9,7 +9,7 @@ export function createBreadcrumb(
   breadcrumb: Pick<Breadcrumb, RequiredProperties> & Partial<Omit<Breadcrumb, RequiredProperties>>,
 ): Breadcrumb {
   return {
-    timestamp: new Date().getTime() / 1000,
+    timestamp: Date.now() / 1000,
     type: 'default',
     ...breadcrumb,
   };

packages/replay/test/fixtures/error.ts

@@ -1,7 +1,7 @@
 import type { Event } from '@sentry/types';
 
 export function Error(obj?: Event): any {
-  const timestamp = new Date().getTime() / 1000;
+  const timestamp = Date.now() / 1000;
 
   return {
     exception: {

packages/replay/test/fixtures/transaction.ts

@@ -1,7 +1,7 @@
 import type { Event, SeverityLevel } from '@sentry/types';
 
 export function Transaction(traceId?: string, obj?: Partial<Event>): any {
-  const timestamp = new Date().getTime() / 1000;
+  const timestamp = Date.now() / 1000;
 
   return {
     contexts: {