commit 4e9f89b0a64395133b9160dcb4533d2dc62a424f
author Leon Scroggins <scroggo@google.com> Mon Feb 22 16:54:14 2010 -0500
committer Leon Scroggins <scroggo@google.com> Tue Feb 23 10:02:58 2010 -0500
tree bb4e02658f77c04937f5cdb829170652f63c9935
parent ab1655e799a3b0a82cea62e05b4017c21ab53881

When long pressing on title bar, do not use WebView in onCreateContextMenu

Fix for http://b/issue?id=2456405

Override TitleBar::createContextMenu to call BrowserActivity's onCreateContextMenu
directly, so that WebView is ignored, and we only get the options for TitleBar.

src/com/android/browser/BrowserActivity.java

diff --git a/src/com/android/browser/BrowserActivity.java b/src/com/android/browser/BrowserActivity.java
index 52a3125..fc337f9 100644
--- a/src/com/android/browser/BrowserActivity.java
+++ b/src/com/android/browser/BrowserActivity.java
@@ -1693,6 +1693,9 @@
     @Override
     public void onCreateContextMenu(ContextMenu menu, View v,
             ContextMenuInfo menuInfo) {
+        if (v instanceof TitleBar) {
+            return;
+        }
         WebView webview = (WebView) v;
         WebView.HitTestResult result = webview.getHitTestResult();
         if (result == null) {

src/com/android/browser/TitleBar.java

diff --git a/src/com/android/browser/TitleBar.java b/src/com/android/browser/TitleBar.java
index 15b1d44..98667c3 100644
--- a/src/com/android/browser/TitleBar.java
+++ b/src/com/android/browser/TitleBar.java
@@ -141,9 +141,10 @@
     };
 
     @Override
-    protected void onCreateContextMenu(ContextMenu menu) {
+    public void createContextMenu(ContextMenu menu) {
         MenuInflater inflater = mBrowserActivity.getMenuInflater();
         inflater.inflate(R.menu.title_context, menu);
+        mBrowserActivity.onCreateContextMenu(menu, this, null);
     }
 
     @Override