upstream: perf: Avoid an extra function call and object clone during …

…event emission (#180)

performance: remove a nested function call and an object clone during
event emission

- rename `event` to `eventWithoutTime`, but maintain backwards
compatibility
- `eventWithTime` (with time) could be renamed to `event` in a future
version

This is an extension of PR rrweb-io#1339 authored by: mydea
<mydea@users.noreply.github.com>

.changeset/event-single-wrap.md

@@ -0,0 +1,5 @@
+---
+'rrweb': patch
+---
+
+perf: Avoid an extra function call and object clone during event emission

packages/rrweb/src/record/iframe-manager.ts

@@ -8,6 +8,7 @@ import CrossOriginIframeMirror from './cross-origin-iframe-mirror';
 import { EventType, IncrementalSource } from '@sentry-internal/rrweb-types';
 import type {
   eventWithTime,
+  eventWithoutTime,
   mutationCallBack,
 } from '@sentry-internal/rrweb-types';
 import type { StylesheetManager } from './stylesheet-manager';
@@ -52,7 +53,7 @@ export class IframeManager implements IframeManagerInterface {
     new WeakMap();
   private mirror: Mirror;
   private mutationCb: mutationCallBack;
-  private wrappedEmit: (e: eventWithTime, isCheckout?: boolean) => void;
+  private wrappedEmit: (e: eventWithoutTime, isCheckout?: boolean) => void;
   private loadListener?: (iframeEl: HTMLIFrameElement) => unknown;
   private stylesheetManager: StylesheetManager;
   private recordCrossOriginIframes: boolean;
@@ -62,7 +63,7 @@ export class IframeManager implements IframeManagerInterface {
     mutationCb: mutationCallBack;
     stylesheetManager: StylesheetManager;
     recordCrossOriginIframes: boolean;
-    wrappedEmit: (e: eventWithTime, isCheckout?: boolean) => void;
+    wrappedEmit: (e: eventWithoutTime, isCheckout?: boolean) => void;
   }) {
     this.mutationCb = options.mutationCb;
     this.wrappedEmit = options.wrappedEmit;