Translate: do not show the context menu entry when the feature is disabled

chrome/browser/renderer_context_menu/render_view_context_menu.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/renderer_context_menu/render_view_context_menu.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <set>
@@ skipping 1227 matching lines @@
     menu_model_.AddItemWithStringId(IDC_SAVE_PAGE,
                                     IDS_CONTENT_CONTEXT_SAVEPAGEAS);
   }
   menu_model_.AddItemWithStringId(IDC_PRINT, IDS_CONTENT_CONTEXT_PRINT);
   AppendMediaRouterItem();
 
   if (TranslateService::IsTranslatableURL(params_.page_url)) {
     std::unique_ptr<translate::TranslatePrefs> prefs(
         ChromeTranslateClient::CreateTranslatePrefs(
             GetPrefs(browser_context_)));
-    std::string locale =
-        translate::TranslateManager::GetTargetLanguage(prefs.get());
-    base::string16 language =
-        l10n_util::GetDisplayNameForLocale(locale, locale, true);
-    menu_model_.AddItem(
-        IDC_CONTENT_CONTEXT_TRANSLATE,
-        l10n_util::GetStringFUTF16(IDS_CONTENT_CONTEXT_TRANSLATE, language));
-    AddGoogleIconToLastMenuItem(&menu_model_);
+    if (prefs->IsEnabled()) {

[Takashi Toyoshima, 2017/04/14 10:22:28]

This is the main change. Others are just a mechanical for file moving.

[lazyboy, 2017/04/17 20:25:45]

Thanks for pointing this one out.

+      std::string locale =
+          translate::TranslateManager::GetTargetLanguage(prefs.get());
+      base::string16 language =
+          l10n_util::GetDisplayNameForLocale(locale, locale, true);
+      menu_model_.AddItem(
+          IDC_CONTENT_CONTEXT_TRANSLATE,
+          l10n_util::GetStringFUTF16(IDS_CONTENT_CONTEXT_TRANSLATE, language));
+      AddGoogleIconToLastMenuItem(&menu_model_);
+    }
   }
 }
 
 void RenderViewContextMenu::AppendExitFullscreenItem() {
   Browser* browser = GetBrowser();
   if (!browser)
     return;
 
   // Only show item if in fullscreen mode.
   if (!browser->exclusive_access_manager()
@@ skipping 1233 matching lines @@
 void RenderViewContextMenu::PluginActionAt(
     const gfx::Point& location,
     const WebPluginAction& action) {
   source_web_contents_->GetRenderViewHost()->
       ExecutePluginActionAtLocation(location, action);
 }
 
 Browser* RenderViewContextMenu::GetBrowser() const {
   return chrome::FindBrowserWithWebContents(embedder_web_contents_);
 }