fix(FEC-10516): Fullscreen event dispatched twice from the player (#489)

kaltura · Sep 30, 2020 · 359bb68 · 359bb68
1 parent 4e4a78f
commit 359bb68
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/fullscreen/fullscreen-controller.js b/src/fullscreen/fullscreen-controller.js
@@ -19,6 +19,8 @@ class FullscreenController {
   _player: Player;
   _isInBrowserFullscreen: boolean;
   _eventManager: EventManager;
+  // Flag to overcome browsers which supports more than one fullscreenchange event
+  _isFullscreenEventDispatched: boolean = false;
 
   /**
    * after component mounted, set up event listeners to window fullscreen state change
@@ -244,7 +246,8 @@ class FullscreenController {
    * @returns {void}
    */
   _fullscreenEnterHandler(): void {
-    if (this.isFullscreen()) {
+    if (this.isFullscreen() && !this._isFullscreenEventDispatched) {
+      this._isFullscreenEventDispatched = true;
       this._player.dispatchEvent(new FakeEvent(this._player.Event.ENTER_FULLSCREEN));
     }
   }
@@ -255,7 +258,8 @@ class FullscreenController {
    * @returns {void}
    */
   _fullscreenExitHandler(): void {
-    if (!this.isFullscreen()) {
+    if (!this.isFullscreen() && this._isFullscreenEventDispatched) {
+      this._isFullscreenEventDispatched = false;
       this._eventManager.removeAll();
       this._player.dispatchEvent(new FakeEvent(this._player.Event.EXIT_FULLSCREEN));
     }