OSDN Git Service

Allow long press MENU to open IME in Browser.
authorLeon Scroggins <scroggo@google.com>
Tue, 8 Dec 2009 15:44:28 +0000 (10:44 -0500)
committerLeon Scroggins <scroggo@google.com>
Tue, 8 Dec 2009 15:44:28 +0000 (10:44 -0500)
Pass MENU key up even if menu is already held down so the
IME can open due to long press.

Fixes http://b/issue?id=20306904

src/com/android/browser/BrowserActivity.java

index 46e6de1..71ed2a1 100644 (file)
@@ -2025,15 +2025,18 @@ public class BrowserActivity extends Activity
 
     @Override
     public boolean onKeyDown(int keyCode, KeyEvent event) {
+        // Even if MENU is already held down, we need to call to super to open
+        // the IME on long press.
+        if (KeyEvent.KEYCODE_MENU == keyCode) {
+            mMenuIsDown = true;
+            return super.onKeyDown(keyCode, event);
+        }
         // The default key mode is DEFAULT_KEYS_SEARCH_LOCAL. As the MENU is
         // still down, we don't want to trigger the search. Pretend to consume
         // the key and do nothing.
         if (mMenuIsDown) return true;
 
         switch(keyCode) {
-            case KeyEvent.KEYCODE_MENU:
-                mMenuIsDown = true;
-                break;
             case KeyEvent.KEYCODE_SPACE:
                 // WebView/WebTextView handle the keys in the KeyDown. As
                 // the Activity's shortcut keys are only handled when WebView