Merge pull request #47 from cloudflare/issue-44-check-if-document-obj…

…ect-before-using Issue 44 - Check if document is defined before using
cloudflare · Jan 7, 2025 · 30c7454 · 30c7454
2 parents 987d903 + 4007768
commit 30c7454
Showing 1 changed file with 20 additions and 7 deletions.
diff --git a/src/engines/BandwidthEngine/BandwidthEngine.js b/src/engines/BandwidthEngine/BandwidthEngine.js
@@ -64,7 +64,12 @@ class BandwidthMeasurementEngine {
     this.#throttleMs = throttleMs;
     this.#estimatedServerTime = Math.max(0, estimatedServerTime);
 
-    document.addEventListener('visibilitychange', this.#handleVisibilityChange);
+    if (typeof document !== 'undefined') {
+      document.addEventListener(
+        'visibilitychange',
+        this.#handleVisibilityChange
+      );
+    }
   }
 
   // Public attributes
@@ -90,9 +95,15 @@ class BandwidthMeasurementEngine {
   }
 
   #handleVisibilityChange = () => {
-    if (document.visibilityState === 'hidden') {
+    if (
+      typeof document !== 'undefined' &&
+      document.visibilityState === 'hidden'
+    ) {
       this.pause();
-    } else if (document.visibilityState === 'visible') {
+    } else if (
+      typeof document !== 'undefined' &&
+      document.visibilityState === 'visible'
+    ) {
       this.play();
     }
   };
@@ -383,10 +394,12 @@ class BandwidthMeasurementEngine {
   }
 
   deleteEventListener() {
-    document.removeEventListener(
-      'visibilitychange',
-      this.#handleVisibilityChange
-    );
+    if (typeof document !== 'undefined') {
+      document.removeEventListener(
+        'visibilitychange',
+        this.#handleVisibilityChange
+      );
+    }
   }
 }