Skip to content

Commit

Permalink
Bugfix/fullscreen promise error (#3669)
Browse files Browse the repository at this point in the history
* Fixed 'Uncaught (in promise)' error when entering or leaving fullscreen.
  • Loading branch information
ShikiSeiren committed Jun 14, 2021
1 parent 1e2fa60 commit d804b04
Showing 1 changed file with 21 additions and 17 deletions.
38 changes: 21 additions & 17 deletions contrib/akamai/controlbar/ControlBar.js
Original file line number Diff line number Diff line change
Expand Up @@ -428,16 +428,18 @@ var ControlBar = function (dashjsMediaPlayer, displayUTCTimeCodes) {

var enterFullscreen = function () {
var element = videoContainer || video;

if (element.requestFullscreen) {
element.requestFullscreen();
} else if (element.msRequestFullscreen) {
element.msRequestFullscreen();
} else if (element.mozRequestFullScreen) {
element.mozRequestFullScreen();
} else {
element.webkitRequestFullScreen();
if(!document.fullscreenElement){
if (element.requestFullscreen) {
element.requestFullscreen();
} else if (element.msRequestFullscreen) {
element.msRequestFullscreen();
} else if (element.mozRequestFullScreen) {
element.mozRequestFullScreen();
} else {
element.webkitRequestFullScreen();
}
}

videoController.classList.add('video-controller-fullscreen');
window.addEventListener('mousemove', onFullScreenMouseMove);
onFullScreenMouseMove();
Expand All @@ -460,16 +462,18 @@ var ControlBar = function (dashjsMediaPlayer, displayUTCTimeCodes) {
clearFullscreenState();

if (document.fullscreenElement) {

if (document.exitFullscreen) {
document.exitFullscreen();
} else if (document.exitFullscreen) {
document.exitFullscreen();
} else if (document.mozCancelFullScreen) {
document.mozCancelFullScreen();
} else if (document.msExitFullscreen) {
document.msExitFullscreen();
} else {
document.webkitCancelFullScreen();
} else if (document.mozCancelFullScreen) {
document.mozCancelFullScreen();
} else if (document.msExitFullscreen) {
document.msExitFullscreen();
} else {
document.webkitCancelFullScreen();
}
}

videoController.classList.remove('video-controller-fullscreen');
};

Expand Down

0 comments on commit d804b04

Please sign in to comment.