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

mydea · 2023-03-16T17:36:48Z

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

github-actions · 2023-03-16T17:48:20Z

size-limit report 📦

Path	Size
@sentry/browser - ES5 CDN Bundle (gzipped + minified)	20.46 KB (-0.14% 🔽)
@sentry/browser - ES5 CDN Bundle (minified)	63.38 KB (-0.18% 🔽)
@sentry/browser - ES6 CDN Bundle (gzipped + minified)	19.07 KB (+0.08% 🔺)
@sentry/browser - ES6 CDN Bundle (minified)	56.43 KB (+0.07% 🔺)
@sentry/browser - Webpack (gzipped + minified)	20.72 KB (+0.09% 🔺)
@sentry/browser - Webpack (minified)	67.65 KB (+0.06% 🔺)
@sentry/react - Webpack (gzipped + minified)	20.74 KB (+0.09% 🔺)
@sentry/nextjs Client - Webpack (gzipped + minified)	52.2 KB (+0.02% 🔺)
@sentry/browser + @sentry/tracing - ES5 CDN Bundle (gzipped + minified)	33.72 KB (-0.13% 🔽)
@sentry/browser + @sentry/tracing - ES6 CDN Bundle (gzipped + minified)	26.1 KB (+0.05% 🔺)
@sentry/replay ES6 CDN Bundle (gzipped + minified)	43.86 KB (-0.03% 🔽)
@sentry/replay - Webpack (gzipped + minified)	37.94 KB (-0.03% 🔽)
@sentry/browser + @sentry/tracing + @sentry/replay - ES6 CDN Bundle (gzipped + minified)	62.06 KB (-0.01% 🔽)
@sentry/browser + @sentry/replay - ES6 CDN Bundle (gzipped + minified)	55.12 KB (0%)

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

0ffd943

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

mydea added the Package: replay Issues related to the Sentry Replay SDK label Mar 16, 2023

mydea requested review from billyvg and Lms24 March 16, 2023 17:36

mydea self-assigned this Mar 16, 2023

billyvg approved these changes Mar 16, 2023

View reviewed changes

mydea merged commit 873d9bb into develop Mar 17, 2023

mydea deleted the fn/date-now branch March 17, 2023 07:44