Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java |
| index 496e81def3560abef6d80be3a63d260d90c79725..b8e79d094f94b395bfec920661307dbc7931dcb4 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java |
| @@ -75,6 +75,10 @@ public class ChromeContextMenuPopulator implements ContextMenuPopulator { |
| menu.findItem(R.id.contextmenu_open_original_image_in_new_tab).setVisible(false); |
| } |
| + // Avoid showing open image option for same image which is already opened. |
| + if (mDelegate.getPageUrl().equals(params.getSrcUrl())) { |
| + menu.findItem(R.id.contextmenu_open_image).setVisible(false); |
| + } |
|
Bernhard Bauer
2014/07/16 13:21:57
This is from a different CL.
|
| final TemplateUrlService templateUrlServiceInstance = TemplateUrlService.getInstance(); |
| final boolean isSearchByImageAvailable = |
| UrlUtilities.isDownloadableScheme(params.getSrcUrl()) && |
| @@ -119,6 +123,8 @@ public class ChromeContextMenuPopulator implements ContextMenuPopulator { |
| mDelegate.onSearchByImageInNewTab(); |
| } else if (itemId == R.id.contextmenu_copy_image) { |
| mDelegate.onSaveImageToClipboard(params.getSrcUrl()); |
| + } else if (itemId == R.id.contextmenu_copy_image_address) { |
| + mDelegate.onSaveToClipboard(params.getSrcUrl(), true); |
| } else { |
| assert false; |
| } |