Do not show "Translate to ..." item in context menu for chrome:// and FileBrowser pages.

chrome/browser/tab_contents/render_view_context_menu.cc

Index: chrome/browser/tab_contents/render_view_context_menu.cc
===================================================================
--- chrome/browser/tab_contents/render_view_context_menu.cc	(revision 98219)
+++ chrome/browser/tab_contents/render_view_context_menu.cc	(working copy)
@@ -69,6 +69,10 @@
 #include "ui/gfx/favicon_size.h"
 #include "webkit/glue/webmenuitem.h"
 
+#ifdef OS_CHROMEOS
+#include "chrome/browser/extensions/file_manager_util.h"
+#endif
+
 using WebKit::WebContextMenuData;
 using WebKit::WebMediaPlayerAction;
 using WebKit::WebURL;
@@ -672,6 +676,19 @@
   }
 }
 
+static bool ShouldShowTranslateItem(GURL page_url) {

[achuithb, 2011/08/25 17:30:48]

Drive-by review with 2 nits. You don't need to wait for my LGTM.

Why not move this function into the anonymous namespace at the top of this file
instead of having it be a static?

Also, it would be preferable if the param was const GURL& page_url to avoid the
unnecessary copy.

[Oleg Eterevsky, 2011/08/25 17:34:42]

Done. Thanks for your comments.

+  if (page_url.SchemeIs("chrome"))
+    return false;
+
+#ifdef OS_CHROMEOS
+  if (page_url.SchemeIs("chrome-extension") &&
+      page_url.DomainIs(kFileBrowserDomain))
+    return false;
+#endif
+
+  return true;
+}
+
 void RenderViewContextMenu::AppendPageItems() {
   menu_model_.AddItemWithStringId(IDC_BACK, IDS_CONTENT_CONTEXT_BACK);
   menu_model_.AddItemWithStringId(IDC_FORWARD, IDS_CONTENT_CONTEXT_FORWARD);
@@ -681,12 +698,15 @@
                                   IDS_CONTENT_CONTEXT_SAVEPAGEAS);
   menu_model_.AddItemWithStringId(IDC_PRINT, IDS_CONTENT_CONTEXT_PRINT);
 
-  std::string locale = g_browser_process->GetApplicationLocale();
-  locale = TranslateManager::GetLanguageCode(locale);
-  string16 language = l10n_util::GetDisplayNameForLocale(locale, locale, true);
-  menu_model_.AddItem(
-      IDC_CONTENT_CONTEXT_TRANSLATE,
-      l10n_util::GetStringFUTF16(IDS_CONTENT_CONTEXT_TRANSLATE, language));
+  if (ShouldShowTranslateItem(params_.page_url)) {
+    std::string locale = g_browser_process->GetApplicationLocale();
+    locale = TranslateManager::GetLanguageCode(locale);
+    string16 language = l10n_util::GetDisplayNameForLocale(locale, locale,
+                                                           true);
+    menu_model_.AddItem(
+        IDC_CONTENT_CONTEXT_TRANSLATE,
+        l10n_util::GetStringFUTF16(IDS_CONTENT_CONTEXT_TRANSLATE, language));
+  }
 
   menu_model_.AddItemWithStringId(IDC_VIEW_SOURCE,
                                   IDS_CONTENT_CONTEXT_VIEWPAGESOURCE);