Merge pull request mozilla#9815 from Snuffleupagus/rm-disableFullscre…

…en-IE-embedded Remove the `disableFullscreen` hack for embedded IE 11 (issue 9585)
movsb · Jun 14, 2018 · 99bb2fc · 99bb2fc
2 parents 7d16c43 + 977547e
commit 99bb2fc
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 18 deletions.
diff --git a/web/app.js b/web/app.js
@@ -553,10 +553,6 @@ let PDFViewerApplication = {
         support = false;
       }
     }
-    if (support && AppOptions.get('disableFullscreen') === true) {
-      support = false;
-    }
-
     return shadow(this, 'supportsFullscreen', support);
   },
 

diff --git a/web/app_options.js b/web/app_options.js
@@ -43,11 +43,6 @@ const defaultOptions = {
     value: '',
     kind: OptionKind.VIEWER,
   },
-  disableFullscreen: {
-    /** @type {boolean} */
-    value: viewerCompatibilityParams.disableFullscreen || false,
-    kind: OptionKind.VIEWER,
-  },
   disableHistory: {
     /** @type {boolean} */
     value: false,

diff --git a/web/viewer_compatibility.js b/web/viewer_compatibility.js
@@ -18,17 +18,8 @@ if (typeof PDFJSDev === 'undefined' || PDFJSDev.test('GENERIC')) {
   const userAgent =
     (typeof navigator !== 'undefined' && navigator.userAgent) || '';
   const isAndroid = /Android/.test(userAgent);
-  const isIE = /Trident/.test(userAgent);
   const isIOS = /\b(iPad|iPhone|iPod)(?=;)/.test(userAgent);
 
-  // Disable fullscreen support for certain problematic configurations.
-  // Support: IE11+ (when embedded).
-  (function checkFullscreenSupport() {
-    if (isIE && window.parent !== window) {
-      compatibilityParams.disableFullscreen = true;
-    }
-  })();
-
   // Limit canvas size to 5 mega-pixels on mobile.
   // Support: Android, iOS
   (function checkCanvasSizeLimitation() {