Index: chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java |
index 7658126ecacc9b7eae8fdecff46b9a163247b593..d818e820b990ba20d604deaa0ce4e626244480ba 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java |
@@ -7,7 +7,6 @@ package org.chromium.chrome.browser.contextmenu; |
import android.view.ContextMenu; |
import android.view.ContextMenu.ContextMenuInfo; |
import android.view.HapticFeedbackConstants; |
-import android.view.Menu; |
import android.view.MenuItem; |
import android.view.MenuItem.OnMenuItemClickListener; |
import android.view.View; |
@@ -84,14 +83,8 @@ public class ContextMenuHelper implements OnCreateContextMenuListener, OnMenuIte |
public void onCreateContextMenu(ContextMenu menu, View v, ContextMenuInfo menuInfo) { |
if (!shouldShowMenu(mCurrentContextMenuParams)) return; |
- if (mCurrentContextMenuParams.isCustomMenu()) { |
- for (int i = 0; i < mCurrentContextMenuParams.getCustomMenuSize(); i++) { |
- menu.add(Menu.NONE, i, Menu.NONE, mCurrentContextMenuParams.getCustomLabelAt(i)); |
- } |
- } else { |
- assert mPopulator != null; |
- mPopulator.buildContextMenu(menu, v.getContext(), mCurrentContextMenuParams); |
- } |
+ assert mPopulator != null; |
+ mPopulator.buildContextMenu(menu, v.getContext(), mCurrentContextMenuParams); |
for (int i = 0; i < menu.size(); i++) { |
menu.getItem(i).setOnMenuItemClickListener(this); |
@@ -100,15 +93,7 @@ public class ContextMenuHelper implements OnCreateContextMenuListener, OnMenuIte |
@Override |
public boolean onMenuItemClick(MenuItem item) { |
- if (mCurrentContextMenuParams.isCustomMenu()) { |
- if (mNativeContextMenuHelper != 0) { |
- final int action = mCurrentContextMenuParams.getCustomActionAt(item.getItemId()); |
- nativeOnCustomItemSelected(mNativeContextMenuHelper, action); |
- } |
- return true; |
- } else { |
- return mPopulator.onItemSelected(this, mCurrentContextMenuParams, item.getItemId()); |
- } |
+ return mPopulator.onItemSelected(this, mCurrentContextMenuParams, item.getItemId()); |
} |
/** |
@@ -120,11 +105,8 @@ public class ContextMenuHelper implements OnCreateContextMenuListener, OnMenuIte |
} |
private boolean shouldShowMenu(ContextMenuParams params) { |
- // Custom menus are handled by this class and do not require a ContextMenuPopulator. |
- return params.isCustomMenu() |
- || (mPopulator != null && mPopulator.shouldShowContextMenu(params)); |
+ return (mPopulator != null && mPopulator.shouldShowContextMenu(params)); |
} |
private native void nativeOnStartDownload(long nativeContextMenuHelper, boolean isLink); |
- private native void nativeOnCustomItemSelected(long nativeContextMenuHelper, int action); |
} |