Changed 'copy link address' to 'copy email address' for mailto schemes

Changed 'copy link address' to 'copy email address' for mailto schemes

When context menu is invoked for mailto schemes context menu should
show 'copy email address' instead of 'copy link address'.

BUG=415459
Committed: https://crrev.com/cac334a1c9fa759ba759610a4cfcd05adedce259
Cr-Commit-Position: refs/heads/master@{#295461}

[ankit, 2014-09-18 10:21:12]

ankit2.kumar@samsung.com changed reviewers:
+ bauerb@chromium.org, tedchoc@chromium.org

[ankit, 2014-09-18 10:21:13]

@Bernhard, @Ted
PTAL

[Bernhard Bauer, 2014-09-18 10:35:32]

https://codereview.chromium.org/583593002/diff/1/chrome/android/java/src/org/...
File
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java
(right):

https://codereview.chromium.org/583593002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java:62:
if (params.getLinkUrl().startsWith("mailto:")) {
Use MailTo.isMailTo()?

https://codereview.chromium.org/583593002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java:138:
} else if (itemId == R.id.contextmenu_copy_email_address) {
Move this up, right after copy_link_address_text?

[ankit, 2014-09-18 10:45:20]

@Bernhard
PTAL new patch.

https://codereview.chromium.org/583593002/diff/1/chrome/android/java/src/org/...
File
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java
(right):

https://codereview.chromium.org/583593002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java:62:
if (params.getLinkUrl().startsWith("mailto:")) {
On 2014/09/18 10:35:32, Bernhard Bauer wrote:
> Use MailTo.isMailTo()?

Done.

https://codereview.chromium.org/583593002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java:138:
} else if (itemId == R.id.contextmenu_copy_email_address) {
On 2014/09/18 10:35:32, Bernhard Bauer wrote:
> Move this up, right after copy_link_address_text?

Done.

[Bernhard Bauer, 2014-09-18 11:31:27]

lgtm

[ankit, 2014-09-18 11:50:08]

On 2014/09/18 11:31:27, Bernhard Bauer wrote:
> lgtm

Thanks.

[ankit, 2014-09-18 11:50:13]

The CQ bit was checked by ankit2.kumar@samsung.com

[commit-bot: I haz the power, 2014-09-18 11:50:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/583593002/20001

[commit-bot: I haz the power, 2014-09-18 12:48:00]

Committed patchset #2 (id:20001) as b31889c7806bf1859f6f1ee0e03feb18f2a42a99

[commit-bot: I haz the power, 2014-09-18 12:48:35]

Patchset 2 (id:??) landed as
https://crrev.com/cac334a1c9fa759ba759610a4cfcd05adedce259
Cr-Commit-Position: refs/heads/master@{#295461}