fix: rrweb recorder may throw error when stopping recording after an iframe becomes cross-origin

packages/rrweb/src/record/index.ts

@@ -617,7 +617,18 @@ function record<T = eventWithTime>(
       );
     }
     return () => {
-      handlers.forEach((h) => h());
+      handlers.forEach((handler) => {
+        try {
+          handler();
+        } catch (error) {
+          /**
+           * This error can occur in a known scenario:
+           * If an iframe is initially same-origin and observed, but later its src attribute is changed to a cross-origin URL,
+           * attempting to execute the handler in the stop record function will throw a "cannot access cross-origin frame" error.
+           * This error is expected and can be safely ignored.
+           */
+        }
+      });
       processedNodeManager.destroy();
       recording = false;
       unregisterErrorHandler();

packages/rrweb/test/record.test.ts

@@ -990,6 +990,21 @@ describe('record', function (this: ISuite) {
 
     await assertSnapshot(ctx.events);
   });
+
+  it('does not throw error when stopping recording after iframe becomes cross-origin', async () => {
+    await ctx.page.evaluate(async () => {
+      const { record } = (window as unknown as IWindow).rrweb;
+      const stopRecord = record({
+        emit: (window as unknown as IWindow).emit,
+      });
+      const iframe = document.createElement('iframe');
+      document.body.appendChild(iframe);
+      await new Promise((resolve) => setTimeout(resolve, 1000));
+      iframe.src = 'https://www.example.com'; // Change the same origin iframe to a cross origin iframe after it's recorded
+      await new Promise((resolve) => setTimeout(resolve, 1000));
+      stopRecord?.();
+    });
+  });
 });
 
 describe('record iframes', function (this: ISuite) {