Fix exit fullscreen viewport scaling issues. Fixes #833 (#845)

MozillaReality · Nov 27, 2018 · f5e522d · f5e522d
1 parent b92a5e6
commit f5e522d
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/NavigationBarWidget.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/NavigationBarWidget.java
@@ -416,6 +416,14 @@ private void exitFullScreenMode() {
             return;
         }
 
+        // We need to add a delay for the exitFullScreen() call to solve some viewport scaling issues,
+        // See https://github.com/MozillaReality/FirefoxReality/issues/833 for more info.
+        getHandler().postDelayed(() -> {
+            if (SessionStore.get().isInFullScreen()) {
+                SessionStore.get().exitFullScreen();
+            }
+        }, 50);
+
         mBrowserWidget.getPlacement().copyFrom(mSizeBeforeFullScreen);
         mWidgetManager.updateWidget(mBrowserWidget);
 
@@ -427,10 +435,6 @@ private void exitFullScreenMode() {
         mWidgetManager.popWorldBrightness(this);
         AnimationHelper.fadeOut(mFullScreenModeContainer, 0, null);
 
-        if (SessionStore.get().isInFullScreen()) {
-            SessionStore.get().exitFullScreen();
-        }
-
         mWidgetManager.setTrayVisible(true);
         closeFloatingMenus();
         mWidgetManager.popWorldBrightness(mBrightnessWidget);