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 6ef4fcdda74e24240866b4ff7f80b046ff20765b..826a925b1839f20df6caa360739231dce5b75cd0 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 |
@@ -38,8 +38,13 @@ public class ChromeContextMenuPopulator implements ContextMenuPopulator { |
@Override |
public void buildContextMenu(ContextMenu menu, Context context, ContextMenuParams params) { |
- if (params.isImage() && !TextUtils.isEmpty(params.getSrcUrl())) |
- menu.setHeaderTitle(params.getSrcUrl()); |
+ if (params.isImage() && !TextUtils.isEmpty(params.getSrcUrl())) { |
+ if (!params.getSrcUrl().startsWith("data:")) { |
Bernhard Bauer
2014/07/21 10:19:28
Hm, we probably shouldn't give the source URL prec
Jitu( very slow this week)
2014/07/21 11:11:35
Yes you are right... this will solve the problem.
Bernhard Bauer
2014/07/21 13:53:39
Wait, are you sure you are not confusing the link
Jitu( very slow this week)
2014/07/23 08:21:46
Yes you are right..
Link URL is "javascript:"
Th
|
+ menu.setHeaderTitle(params.getSrcUrl()); |
+ } else { |
+ menu.setHeaderTitle(params.getLinkUrl()); |
+ } |
+ } |
else if (!TextUtils.isEmpty(params.getLinkUrl()) ) |
menu.setHeaderTitle(params.getLinkUrl()); |