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

[rginda, 2011/08/25 20:10:16]

The file manager has its own FILE_MANAGER_EXTENSION #define, so that it may one
day be included in non-chromeos builds.

[Oleg Eterevsky, 2011/08/26 11:08:39]

Fixed. Thanks.

+#include "chrome/browser/extensions/file_manager_util.h"
+#endif
+
 using WebKit::WebContextMenuData;
 using WebKit::WebMediaPlayerAction;
 using WebKit::WebURL;
@@ -164,6 +168,19 @@
   }
 }
 
+bool ShouldShowTranslateItem(const GURL& page_url) {
+  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;
+}
+
 }  // namespace
 
 // static
@@ -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);