Fix the problem that wrong context menu shows up when right clicking a link on Mac

chrome/browser/tab_contents/render_view_context_menu.cc

Index: chrome/browser/tab_contents/render_view_context_menu.cc
diff --git a/chrome/browser/tab_contents/render_view_context_menu.cc b/chrome/browser/tab_contents/render_view_context_menu.cc
index 608688f81e8b17ad7b18ef6d44c961b2968f87d9..1bbee50556c326038b2372a7a342857fa93021b1 100644
--- a/chrome/browser/tab_contents/render_view_context_menu.cc
+++ b/chrome/browser/tab_contents/render_view_context_menu.cc
@@ -726,10 +726,12 @@ void RenderViewContextMenu::AppendPanelItems() {
 
   if (params_.is_editable)
     AppendEditableItems();
-  else if (has_selection)
-    AppendCopyItem();
+  // Checking link should take precedence before checking selection since on Mac
+  // right-clicking a link will also make it selected.
   else if (params_.unfiltered_link_url.is_valid())
     AppendLinkItems();
+  else if (has_selection)

[Dmitry Titov, 2013/06/19 01:14:41]

why have an 'else' here? On normal pages we show both Link items and "Copy"...

[jianli, 2013/06/19 17:08:18]

Changed to allow both link and copy.

+    AppendCopyItem();
 
   // Only add extension items from this extension.
   int index = 0;