fix: Clears resources when api will be disposed.

jitsi · Jan 17, 2025 · 6724c78 · 6724c78
1 parent f99e45d
commit 6724c78
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/alwaysontop/render/index.js b/alwaysontop/render/index.js
@@ -58,11 +58,12 @@ class AlwaysOnTop extends EventEmitter {
 
         this._api.on('videoConferenceJoined', this._onConferenceJoined);
         this._api.on('readyToClose', this._disposeWindow);
+        this._api.on('_willDispose', this._disposeWindow);
 
         if (showOnPrejoin) {
             this._api.on('prejoinScreenLoaded', this._onConferenceJoined);
         }
-        
+
         window.parent.addEventListener('beforeunload', this._disposeWindow);
     }
 
@@ -136,7 +137,7 @@ class AlwaysOnTop extends EventEmitter {
             if (!this._aotWindow) {
                 return;
             }
-    
+
             if (!this._jitsiMeetLargeVideo) {
                 this._aotWindowVideo.style.display = 'none';
                 this._aotWindowVideo.srcObject = null;
@@ -240,7 +241,7 @@ class AlwaysOnTop extends EventEmitter {
 
     /**
      * Disposes the aot window
-     * 
+     *
      */
      _disposeWindow() {
         logInfo('disposing window');

diff --git a/powermonitor/render.js b/powermonitor/render.js
@@ -78,6 +78,8 @@ function dispose() {
 module.exports = function setupPowerMonitorRender(api) {
     const iframe = api.getIFrame();
 
+    api.on('_willDispose', dispose);
+
     iframe.addEventListener('load', () => {
         iframe.contentWindow.addEventListener(
             'unload',