feat: Reduce noise from mousemove events (#1424)

ptang-nr · web-flow · commit 7c02a02763ab · 2025-03-27T14:54:12.000-07:00
diff --git a/src/features/session_trace/aggregate/trace/storage.js b/src/features/session_trace/aggregate/trace/storage.js
@@ -15,7 +15,7 @@ const SUPPORTS_PERFORMANCE_OBSERVER = typeof globalScope.PerformanceObserver ===
 
 const ignoredEvents = {
   // we find that certain events make the data too noisy to be useful
-  global: { mouseup: true, mousedown: true },
+  global: { mouseup: true, mousedown: true, mousemove: true },
   // certain events are present both in the window and in PVT metrics.  PVT metrics are prefered so the window events should be ignored
   window: { load: true, pagehide: true },
   // when ajax instrumentation is disabled, all XMLHttpRequest events will return with origin = xhrOriginMissing and should be ignored
@@ -182,8 +182,8 @@ export class TraceStorage {
   }
 
   shouldIgnoreEvent (event, target) {
-    const origin = eventOrigin(event.target, target, this.parent.ee)
     if (event.type in ignoredEvents.global) return true
+    const origin = eventOrigin(event.target, target, this.parent.ee)
     if (!!ignoredEvents[origin] && ignoredEvents[origin].ignoreAll) return true
     return !!(!!ignoredEvents[origin] && event.type in ignoredEvents[origin])
   }
diff --git a/tests/assets/event-listener-mousemove.html b/tests/assets/event-listener-mousemove.html
@@ -0,0 +1,40 @@
+<!DOCTYPE html>
+<!--
+  Copyright 2020 New Relic Corporation.
+  PDX-License-Identifier: Apache-2.0
+-->
+<html lang="en">
+  <head>
+    <title>RUM Unit Test</title>
+    {init}
+    {config}
+    {loader}
+    <script>
+      addEventListener("mousemove", function() {
+        console.log("Mouse moved")
+      })
+    </script>
+  </head>
+  <body>
+    <div id="initial">initial content</div>
+
+    <div id="foobar">
+      FOOBAR!
+    </div>
+    <script>
+        // Create a new mouse event
+        const mouseMoveEvent = new MouseEvent('mousemove', {
+          clientX: 100, // X coordinate relative to the viewport
+          clientY: 200, // Y coordinate relative to the viewport
+          bubbles: true, // Allow the event to bubble up the DOM tree
+          cancelable: true // Allow the event to be canceled
+        })
+
+        // Get the element to dispatch the event to
+        const element = document.getElementById('foobar')
+
+        // Dispatch the event
+        element.dispatchEvent(mouseMoveEvent)
+    </script>
+  </body>
+</html>
diff --git a/tests/specs/session-trace/trace-nodes.e2e.js b/tests/specs/session-trace/trace-nodes.e2e.js
@@ -126,6 +126,20 @@ describe('Trace nodes', () => {
       }, {})
     expect(Object.values(eventCounts).some(count => count > 1)).toBeFalsy()
   })
+
+  it('are not created for mouseover events', async () => {
+    const url = await browser.testHandle.assetURL('event-listener-mousemove.html', stConfig())
+    await browser.url(url).then(() => browser.waitForAgentLoad())
+
+    const [sessionTraceHarvests] = await Promise.all([
+      sessionTraceCapture.waitForResult({ timeout: 10000 })
+    ])
+
+    sessionTraceHarvests.forEach(harvest => {
+      const foobarMousemoveEvts = JSONPath({ path: '$.request.body.[?(!!@ && @.t===\'event\' && @.n===\'mousing\' && @.o===\'div#foobar\')]', json: harvest })
+      expect(foobarMousemoveEvts.length).toEqual(0)
+    })
+  })
 })
 
 function getEventsSetSize () {
