DragEvent handling, null check fixes (#630)

packages/rrweb/src/record/mutation.ts

@@ -271,7 +271,10 @@ export default class MutationBuffer {
       const shadowHost: Element | null = n.getRootNode
         ? (n.getRootNode() as ShadowRoot)?.host
         : null;
-      const notInDoc = !this.doc.contains(n) && !this.doc.contains(shadowHost);
+      // ensure shadowHost is a Node, or doc.contains will throw an error
+      const notInDoc =
+        !this.doc.contains(n) &&
+        (!(shadowHost instanceof Node) || !this.doc.contains(shadowHost));
       if (!n.parentNode || notInDoc) {
         return;
       }
@@ -359,13 +362,16 @@ export default class MutationBuffer {
       if (!node) {
         for (let index = addList.length - 1; index >= 0; index--) {
           const _node = addList.get(index)!;
-          const parentId = this.mirror.getId(
+          // ensure _node is defined before attempting to find value
-            (_node.value.parentNode as Node) as INode,
+          if (_node) {
-          );
+            const parentId = this.mirror.getId(
-          const nextId = getNextId(_node.value);
+              (_node.value.parentNode as Node) as INode,
-          if (parentId !== -1 && nextId !== -1) {
+            );
-            node = _node;
+            const nextId = getNextId(_node.value);
-            break;
+            if (parentId !== -1 && nextId !== -1) {
+              node = _node;
+              break;
+            }
           }
         }
       }

packages/rrweb/src/record/observer.ts

@@ -194,21 +194,43 @@ function initMoveObserver(
     },
     callbackThreshold,
   );
-  const updatePosition = throttle<MouseEvent | TouchEvent | DragEvent>(
+
+  // update position for mouse, touch, and drag events (drag event extends mouse event)
+  function handleUpdatePositionEvent(evt: MouseEvent | TouchEvent) {
+    const target = getEventTarget(evt);
+    const { clientX, clientY } = isTouchEvent(evt)
+      ? evt.changedTouches[0]
+      : evt;
+    if (!timeBaseline) {
+      timeBaseline = Date.now();
+    }
+    positions.push({
+      x: clientX,
+      y: clientY,
+      id: mirror.getId(target as INode),
+      timeOffset: Date.now() - timeBaseline,
+    });
+  }
+
+  // separate call for non-drag events, in case DragEvent is not defined
+  const updatePosition = throttle<MouseEvent | TouchEvent>(
     (evt) => {
-      const target = getEventTarget(evt);
+      handleUpdatePositionEvent(evt);
-      const { clientX, clientY } = isTouchEvent(evt)
+      wrappedCb(
-        ? evt.changedTouches[0]
+        evt instanceof MouseEvent
-        : evt;
+          ? IncrementalSource.MouseMove
-      if (!timeBaseline) {
+          : IncrementalSource.TouchMove,
-        timeBaseline = Date.now();
+      );
-      }
+    },
-      positions.push({
+    threshold,
-        x: clientX,
+    {
-        y: clientY,
+      trailing: false,
-        id: mirror.getId(target as INode),
+    },
-        timeOffset: Date.now() - timeBaseline,
+  );
-      });
+  // call for drag events, when DragEvent is defined
+  const updateDragPosition = throttle<MouseEvent | TouchEvent | DragEvent>(
+    (evt) => {
+      handleUpdatePositionEvent(evt);
       wrappedCb(
         evt instanceof DragEvent
           ? IncrementalSource.Drag
@@ -222,10 +244,13 @@ function initMoveObserver(
       trailing: false,
     },
   );
+  // it is possible DragEvent is undefined even on devices
+  // that support event 'drag'
+  const dragEventDefined = typeof DragEvent !== 'undefined';
   const handlers = [
     on('mousemove', updatePosition, doc),
     on('touchmove', updatePosition, doc),
-    on('drag', updatePosition, doc),
+    on('drag', dragEventDefined ? updateDragPosition : updatePosition, doc),
   ];
   return () => {
     handlers.forEach((h) => h());