Prevent unnecessary creation of MutationObservers

Moved inspectMediaElements to the constructor where it dynamically updates this.elements
vantezzen · Mar 9, 2023 · f2e29c4 · f2e29c4
1 parent 51e2e9c
commit f2e29c4
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/contents/lib/SpeedController.ts b/src/contents/lib/SpeedController.ts
@@ -17,6 +17,12 @@ export default class SpeedController {
 
   elements: MediaElement[] = []
 
+  constructor() {
+    inspectMediaElements((element: MediaElement) => {
+      this.elements.push(element)
+    })
+  }
+
   /**
    * Fixes issues changing the playback rate by temporarily blocking `ratechange` event listeners.
    */
@@ -101,7 +107,7 @@ export default class SpeedController {
   }
 
   setPlaybackRate(rate: number) {
-    inspectMediaElements((element: MediaElement) => {
+    this.elements.forEach((element) => {
       this.setPlaybackRateForElement(rate, element)
     })
   }