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 0cdd16fe1a379febe06a07653e948a607b643260..593159630db62943ef10d60e96f4ceedc3212d6c 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 |
| @@ -5,6 +5,7 @@ |
| package org.chromium.chrome.browser.contextmenu; |
| import android.content.Context; |
| +import android.net.MailTo; |
| import android.os.Build; |
| import android.text.TextUtils; |
| import android.view.ContextMenu; |
| @@ -58,6 +59,12 @@ public class ChromeContextMenuPopulator implements ContextMenuPopulator { |
| menu.findItem(R.id.contextmenu_copy_link_text).setVisible(false); |
| } |
| + if (params.getLinkUrl().startsWith("mailto:")) { |
|
Bernhard Bauer
2014/09/18 10:35:32
Use MailTo.isMailTo()?
ankit
2014/09/18 10:45:20
Done.
|
| + menu.findItem(R.id.contextmenu_copy_link_address_text).setVisible(false); |
| + } else { |
| + menu.findItem(R.id.contextmenu_copy_email_address).setVisible(false); |
| + } |
| + |
| menu.findItem(R.id.contextmenu_save_link_as).setVisible( |
| UrlUtilities.isDownloadableScheme(params.getLinkUrl())); |
| @@ -128,6 +135,8 @@ public class ChromeContextMenuPopulator implements ContextMenuPopulator { |
| mDelegate.onSaveImageToClipboard(params.getSrcUrl()); |
| } else if (itemId == R.id.contextmenu_copy_image_url) { |
| mDelegate.onSaveToClipboard(params.getSrcUrl(), true); |
| + } else if (itemId == R.id.contextmenu_copy_email_address) { |
|
Bernhard Bauer
2014/09/18 10:35:32
Move this up, right after copy_link_address_text?
ankit
2014/09/18 10:45:20
Done.
|
| + mDelegate.onSaveToClipboard(MailTo.parse(params.getLinkUrl()).getTo(), false); |
| } else { |
| assert false; |
| } |