Add fullscreen toggle to menu

Change-Id: I7acecfc32dde250685f96354b45a91b93667114e

6 files changed, 28 insertions, 9 deletions

diff --git a/src/com/android/browser/BaseUi.java b/src/com/android/browser/BaseUi.java
index bfee02f..db8f0e1 100644
--- a/src/com/android/browser/BaseUi.java
+++ b/src/com/android/browser/BaseUi.java
@@ -139,7 +139,7 @@ public abstract class BaseUi implements UI {
                 R.id.fullscreen_custom_content);
         mErrorConsoleContainer = (LinearLayout) frameLayout
                 .findViewById(R.id.error_console);
-        setFullscreen(BrowserSettings.getInstance().useFullscreen());
+        setFullscreen(false);
         mGenericFavicon = res.getDrawable(
                 R.drawable.app_web_browser_sm);
         mTitleBar = new TitleBar(mActivity, mUiController, this,
@@ -790,6 +790,13 @@ public abstract class BaseUi implements UI {
         win.setAttributes(winParams);
     }
 
+    public boolean isFullscreen() {
+        Window win = mActivity.getWindow();
+        WindowManager.LayoutParams winParams = win.getAttributes();
+        final int bits = WindowManager.LayoutParams.FLAG_FULLSCREEN;
+        return (winParams.flags & bits) == bits;
+    }
+
     public Drawable getFaviconDrawable(Bitmap icon) {
         Drawable[] array = new Drawable[3];
         array[0] = new PaintDrawable(Color.BLACK);
diff --git a/src/com/android/browser/BrowserSettings.java b/src/com/android/browser/BrowserSettings.java
index 2a88d7d..9a0e74c 100644
--- a/src/com/android/browser/BrowserSettings.java
+++ b/src/com/android/browser/BrowserSettings.java
@@ -376,10 +376,6 @@ public class BrowserSettings implements OnSharedPreferenceChangeListener,
         syncManagedSettings();
         if (PREF_SEARCH_ENGINE.equals(key)) {
             updateSearchEngine(false);
-        } else if (PREF_FULLSCREEN.equals(key)) {
-            if (mController.getUi() != null) {
-                mController.getUi().setFullscreen(useFullscreen());
-            }
         } else if (PREF_ENABLE_QUICK_CONTROLS.equals(key)) {
             if (mController.getUi() != null) {
                 mController.getUi().setUseQuickControls(sharedPreferences.getBoolean(key, false));
@@ -829,10 +825,6 @@ public class BrowserSettings implements OnSharedPreferenceChangeListener,
         return HomeProvider.MOST_VISITED.equals(getHomePage());
     }
 
-    public boolean useFullscreen() {
-        return mPrefs.getBoolean(PREF_FULLSCREEN, false);
-    }
-
     public boolean useInvertedRendering() {
         return mPrefs.getBoolean(PREF_INVERTED, false);
     }
diff --git a/src/com/android/browser/Controller.java b/src/com/android/browser/Controller.java
index ee55f6d..108a613 100644
--- a/src/com/android/browser/Controller.java
+++ b/src/com/android/browser/Controller.java
@@ -1593,6 +1593,10 @@ public class Controller
         boolean showDebugSettings = mSettings.isDebugEnabled();
         final MenuItem uaSwitcher = menu.findItem(R.id.ua_desktop_menu_id);
         uaSwitcher.setChecked(isDesktopUa);
+
+        final MenuItem fullscreen = menu.findItem(R.id.fullscreen_menu_id);
+        fullscreen.setChecked(mUi.isFullscreen());
+
         menu.setGroupVisible(R.id.LIVE_MENU, isLive);
         menu.setGroupVisible(R.id.SNAPSHOT_MENU, !isLive);
         menu.setGroupVisible(R.id.COMBO_MENU, false);
@@ -1730,6 +1734,9 @@ public class Controller
                 toggleUserAgent();
                 break;
 
+            case R.id.fullscreen_menu_id:
+                toggleFullscreen();
+
             case R.id.window_one_menu_id:
             case R.id.window_two_menu_id:
             case R.id.window_three_menu_id:
@@ -1830,6 +1837,11 @@ public class Controller
     }
 
     @Override
+    public void toggleFullscreen() {
+        mUi.setFullscreen(!mUi.isFullscreen());
+    }
+
+    @Override
     public void findOnPage() {
         getCurrentTopWebView().showFindDialog(null, true);
     }
diff --git a/src/com/android/browser/PhoneUi.java b/src/com/android/browser/PhoneUi.java
index e3c22bd..dc3b863 100644
--- a/src/com/android/browser/PhoneUi.java
+++ b/src/com/android/browser/PhoneUi.java
@@ -182,6 +182,10 @@ public class PhoneUi extends BaseUi {
         if (incognito != null) {
             incognito.setVisible(showingNavScreen() || mUseQuickControls);
         }
+        MenuItem fullscreen = menu.findItem(R.id.fullscreen_menu_id);
+        if (fullscreen != null) {
+            fullscreen.setVisible(!showingNavScreen());
+        }
         MenuItem closeOthers = menu.findItem(R.id.close_other_tabs_id);
         if (closeOthers != null) {
             boolean isLastTab = true;
diff --git a/src/com/android/browser/UI.java b/src/com/android/browser/UI.java
index 677c3d3..d9bb3bc 100644
--- a/src/com/android/browser/UI.java
+++ b/src/com/android/browser/UI.java
@@ -142,6 +142,8 @@ public interface UI {
 
     void setFullscreen(boolean enabled);
 
+    boolean isFullscreen();
+
     void setUseQuickControls(boolean enabled);
 
     public boolean shouldCaptureThumbnails();
diff --git a/src/com/android/browser/UiController.java b/src/com/android/browser/UiController.java
index 683e473..494a04f 100644
--- a/src/com/android/browser/UiController.java
+++ b/src/com/android/browser/UiController.java
@@ -107,6 +107,8 @@ public interface UiController {
 
     void toggleUserAgent();
 
+    void toggleFullscreen();
+
     BrowserSettings getSettings();
 
     boolean supportsVoice();