Show both custom and default menu items when selection is present.

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 e21ff5c7b12ac06889e1f5eac0ae91817a0aebb8..412e8a248fe7275fa38c04caabc6a4c27fcc6baf 100644
--- a/chrome/browser/tab_contents/render_view_context_menu.cc
+++ b/chrome/browser/tab_contents/render_view_context_menu.cc
@@ -509,10 +509,14 @@ void RenderViewContextMenu::InitMenu() {
   bool has_selection = !params_.selection_text.empty();
 
   if (AppendCustomItems()) {
-    // Don't add items for Pepper menu.
-    if (!params_.custom_context.is_pepper_menu)
-      AppendDeveloperItems();
-    return;
+    if (has_selection) {
+      menu_model_.AddSeparator();

[brettw, 2011/10/21 20:24:27]

I get it. Can we have a comment here like:
  // If there's a selection, don't early return when there are custom items, but
fall through to adding the normal ones after the custom ones.

+    } else {
+      // Don't add items for Pepper menu.
+      if (!params_.custom_context.is_pepper_menu)
+        AppendDeveloperItems();
+      return;
+    }
   }
 
   // When no special node or text is selected and selection has no link,