diff options
| author | ankit2.kumar@samsung.com <ankit2.kumar@samsung.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-17 11:05:16 +0000 |
|---|---|---|
| committer | ankit2.kumar@samsung.com <ankit2.kumar@samsung.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-17 11:05:16 +0000 |
| commit | ca91045e993740f61401a5ae0ac0b8b24fa826b0 (patch) | |
| tree | f08dabcee6839c32762f73eb7f9373b22b8a8c28 | |
| parent | 1daebe7686bc4e743a0c7e1b35c71761fb04e68b (diff) | |
| download | chromium_src-ca91045e993740f61401a5ae0ac0b8b24fa826b0.zip chromium_src-ca91045e993740f61401a5ae0ac0b8b24fa826b0.tar.gz chromium_src-ca91045e993740f61401a5ae0ac0b8b24fa826b0.tar.bz2 | |
Context menu for image is not having any option to copy image URL.
There is no direct way to copy image url through context menu by
which user can copy image url. Added an option in context menu
for image to copy image url.
BUG=394314
Review URL: https://codereview.chromium.org/398803002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283746 0039d316-1c4b-4281-b951-d872f2087c98
3 files changed, 7 insertions, 0 deletions
diff --git a/chrome/android/java/res/menu/chrome_context_menu.xml b/chrome/android/java/res/menu/chrome_context_menu.xml index 055cd2a..abeeea2 100644 --- a/chrome/android/java/res/menu/chrome_context_menu.xml +++ b/chrome/android/java/res/menu/chrome_context_menu.xml @@ -32,6 +32,8 @@ tools:ignore="MenuTitle" /> <item android:id="@+id/contextmenu_copy_image" android:title="@string/contextmenu_copy_image"/> + <item android:id="@+id/contextmenu_copy_image_url" + android:title="@string/contextmenu_copy_image_url"/> </group> <group android:id="@+id/contextmenu_group_video"> <item android:id="@+id/contextmenu_save_video" 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 6ef4fcd..66d47af 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 @@ -123,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_url) { + mDelegate.onSaveToClipboard(params.getSrcUrl(), true); } else { assert false; } diff --git a/chrome/android/java/strings/android_chrome_strings.grd b/chrome/android/java/strings/android_chrome_strings.grd index 658c023..18e8251 100644 --- a/chrome/android/java/strings/android_chrome_strings.grd +++ b/chrome/android/java/strings/android_chrome_strings.grd @@ -302,6 +302,9 @@ You are signing in with a managed account and giving its administrator control o <message name="IDS_CONTEXTMENU_COPY_IMAGE" desc="Context sensitive menu item for copying an image to the clipboard. [CHAR-LIMIT=30]"> Copy image </message> + <message name="IDS_CONTEXTMENU_COPY_IMAGE_URL" desc="Context sensitive menu item for copying an image url to the clipboard. [CHAR-LIMIT=30]"> + Copy image URL + </message> <message name="IDS_CONTEXTMENU_SAVE_VIDEO" desc="Context sensitive menu item for saving the selected video. [CHAR-LIMIT=30]"> Save video </message> |