commit 3ca12750f9b6231b00a1856dcb25c2d2e07e8784
author Michael Kolb <kolby@google.com> Wed Jul 20 13:52:25 2011 -0700
committer Michael Kolb <kolby@google.com> Wed Jul 20 13:52:29 2011 -0700
tree 155cd2b6c7fa60385863427147a1e9a7d6472cb4
parent 3b02d34b1f3c6c1629f23b9ffcb372fbf2f1d880

use system menu

    On phones with hardware menu key, use the framework's
    menu
    Also fixes:
    Bug: 5051899
    remove overflow button from bar if menu key is present
    Nav options are not visible on tab switcher

Change-Id: I5b55934a392a76f9d9ef16e3bf5c71f27b88d69a

src/com/android/browser/PhoneUi.java

diff --git a/src/com/android/browser/PhoneUi.java b/src/com/android/browser/PhoneUi.java
index 9e3310b..28db6f0 100644
--- a/src/com/android/browser/PhoneUi.java
+++ b/src/com/android/browser/PhoneUi.java
@@ -23,6 +23,7 @@
 import android.view.Gravity;
 import android.view.KeyEvent;
 import android.view.Menu;
+import android.view.MenuItem;
 import android.view.View;
 import android.webkit.WebView;
 import android.widget.FrameLayout;
@@ -78,16 +79,6 @@
     }
 
     @Override
-    public boolean onMenuKey() {
-        if (mNavigationBar.isMenuShowing()) {
-            mNavigationBar.dismissMenu();
-        } else {
-            mNavigationBar.showMenu();
-        }
-        return true;
-    }
-
-    @Override
     public boolean dispatchKey(int code, KeyEvent event) {
         return false;
     }
@@ -169,6 +160,20 @@
     // menu handling callbacks
 
     @Override
+    public boolean onPrepareOptionsMenu(Menu menu) {
+        menu.setGroupVisible(R.id.NAV_MENU, (mNavScreen == null));
+        return true;
+    }
+
+    @Override
+    public boolean onOptionsItemSelected(MenuItem item) {
+        if (mNavScreen != null) {
+            hideNavScreen(false);
+        }
+        return false;
+    }
+
+    @Override
     public void onContextMenuCreated(Menu menu) {
         hideTitleBar();
     }