Safer Media play/pause when target is not in fact a MediaElement

.changeset/gorgeous-eels-rescue.md

@@ -0,0 +1,5 @@
+---
+"rrweb": patch
+---
+
+Safer Media play/pause when target is not in fact a MediaElement

packages/rrweb/src/record/observer.ts

@@ -1027,6 +1027,7 @@ function initMediaInteractionObserver({
         const target = getEventTarget(event);
         if (
           !target ||
+          !(target instanceof HTMLMediaElement) ||
           isBlocked(target as Node, blockClass, blockSelector, true)
         ) {
           return;

packages/rrweb/src/replay/media/index.ts

@@ -54,7 +54,14 @@ export class MediaManager {
     this.mediaMap.forEach((_mediaState, target) => {
       this.syncTargetWithState(target);
       if (options.pause) {
-        target.pause();
+        try {
+          target.pause();
+        } catch (error) {
+          this.warn(
+            // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/restrict-template-expressions
+            `Failed to sync media element: ${error.message || error}`,
+          );
+        }
       }
     });
   }
@@ -104,7 +111,14 @@ export class MediaManager {
 
       target.currentTime = seekToTime;
     } else {
-      target.pause();
+      try {
+        target.pause();
+      } catch (error) {
+        this.warn(
+          // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/restrict-template-expressions
+          `Failed to pause during seek: ${error.message || error}`,
+        );
+      }
       target.currentTime = mediaState.currentTimeAtLastInteraction;
     }
   }