fix(App): Fix potential memory leak with full screen event listeners

Author: adlk

src/electron/ipc-api/fullscreen.js

@@ -0,0 +1,10 @@
+export const UPDATE_FULL_SCREEN_STATUS = 'set-full-screen-status';
+
+export default ({ mainWindow }) => {
+  mainWindow.on('enter-full-screen', () => {
+    mainWindow.webContents.send(UPDATE_FULL_SCREEN_STATUS, true);
+  });
+  mainWindow.on('leave-full-screen', () => {
+    mainWindow.webContents.send(UPDATE_FULL_SCREEN_STATUS, false);
+  });
+};

src/electron/ipc-api/index.js

@@ -7,6 +7,7 @@ import dnd from './dnd';
 import desktopCapturer from './desktopCapturer';
 import focusState from './focusState';
 import serviceWebView from '../../features/serviceWebview/ipc';
+import fullscreenStatus from './fullscreen';
 
 export default (params) => {
   settings(params);
@@ -18,4 +19,5 @@ export default (params) => {
   desktopCapturer();
   focusState(params);
   serviceWebView(params);
+  fullscreenStatus(params);
 };

src/stores/AppStore.js

@@ -25,6 +25,7 @@ import { getLocale } from '../helpers/i18n-helpers';
 import { getServiceIdsFromPartitions, removeServicePartitionDirectory } from '../helpers/service-helpers.js';
 import { isValidExternalURL } from '../helpers/url-helpers';
 import { sleep } from '../helpers/async-helpers';
+import { UPDATE_FULL_SCREEN_STATUS } from '../electron/ipc-api/fullscreen';
 
 const debug = require('debug')('Franz:AppStore');
 
@@ -118,13 +119,9 @@ export default class AppStore extends Store {
       this.isOnline = false;
     });
 
-    mainWindow.on('enter-full-screen', () => {
-      this.isFullScreen = true;
+    document.addEventListener('onfullscreenchange', () => {
+      console.log('fullscreen', document.fullscreenEnabled);
     });
-    mainWindow.on('leave-full-screen', () => {
-      this.isFullScreen = false;
-    });
-
 
     this.isOnline = navigator.onLine;
 
@@ -178,6 +175,10 @@ export default class AppStore extends Store {
       }
     });
 
+    ipcRenderer.on(UPDATE_FULL_SCREEN_STATUS, (e, status) => {
+      this.isFullScreen = status;
+    });
+
     // Handle deep linking (franz://)
     ipcRenderer.on('navigateFromDeepLink', (event, data) => {
       debug('Navigate from deep link', data);