Merge pull request #16 from pendo-io/meg-define-take-full-snapshot-be…

…fore-use define takeFullSnapshot before use in iframeManager constructor to prevent undefined error
pendo-io · May 16, 2024 · c504a11 · c504a11
2 parents 168e606 + 86b9b3b
commit c504a11
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/packages/rrweb/src/record/iframe-manager.ts b/packages/rrweb/src/record/iframe-manager.ts
@@ -50,6 +50,10 @@ export class IframeManager {
     }
   }
 
+  public setTakeFullSnapshot(takeFullSnapshot: (isCheckout?: boolean) => void) {
+    this.takeFullSnapshot = takeFullSnapshot;
+  }
+
   public addIframe(iframeEl: HTMLIFrameElement) {
     this.iframes.set(iframeEl, true);
     if (iframeEl.contentWindow)

diff --git a/packages/rrweb/src/record/index.ts b/packages/rrweb/src/record/index.ts
@@ -41,8 +41,9 @@ import {
 } from './error-handler';
 
 let wrappedEmit!: (e: eventWithoutTime, isCheckout?: boolean) => void;
-
-let takeFullSnapshot!: (isCheckout?: boolean) => void;
+let takeFullSnapshot: (isCheckout?: boolean) => void = () => {
+  /* no-op */
+};
 let canvasManager!: CanvasManager;
 let recording = false;
 
@@ -418,6 +419,7 @@ function record<T = eventWithTime>(
         mirror.getId(document),
       );
   };
+  iframeManager.setTakeFullSnapshot(takeFullSnapshot);
 
   try {
     const handlers: listenerHandler[] = [];