Revert "Consolidate translation check and execution logic between...

Revert "Consolidate translation check and execution logic between TranslateManager and RenderViewContextMenu"

This reverts commit 24c91f76.

Reason for revert: Resulting browser process crashes

Original change's description:
> Consolidate translation check and execution logic between TranslateManager and RenderViewContextMenu
> 
> Consolidated logic by having RenderViewContextMenu::ExecTranslate call
> TranslateManager::InitiateManualTranslation, and RenderViewContextMenu::AppendPageItems call
> TranslateManager:CanManuallyTranslate.
> 
> Bug: 898168
> Change-Id: If91cf98626f73d0c357522f8fdca840695185bc2
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2385636
> Reviewed-by: Avi Drissman <avi@chromium.org>
> Reviewed-by: Megan Jablonski <megjablon@chromium.org>
> Commit-Queue: Anthony Cui <cuianthony@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#806161}

TBR=avi@chromium.org,sclittle@chromium.org,megjablon@chromium.org,cuianthony@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 898168
Change-Id: I23e202b72625365eedcd124af000c972ced3f816
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2409140Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Megan Jablonski <megjablon@chromium.org>
Commit-Queue: Anthony Cui <cuianthony@chromium.org>
Cr-Commit-Position: refs/heads/master@{#806664}

components/translate/core/browser/translate_manager.cc

@@ -200,9 +200,6 @@ bool TranslateManager::CanManuallyTranslate() {
 
   std::unique_ptr<TranslatePrefs> translate_prefs(
       translate_client_->GetTranslatePrefs());
-  if (!translate_prefs->IsTranslateAllowedByPolicy())
-    return false;
-  
   const std::string target_lang = GetManualTargetLanguage(
       TranslateDownloadManager::GetLanguageCode(source_language),
       language_state_, translate_prefs.get(), language_model_);
@@ -212,13 +209,7 @@ bool TranslateManager::CanManuallyTranslate() {
   return true;
 }
 
-void TranslateManager::InitiateManualTranslation(bool auto_translate,
-                                                 bool triggered_from_menu) {
-  // If a translation has already been triggered, do nothing.
-  if (language_state_.IsPageTranslated() ||
-      language_state_.translation_pending())
-    return;
-
+void TranslateManager::InitiateManualTranslation(bool auto_translate) {
   std::unique_ptr<TranslatePrefs> translate_prefs(
       translate_client_->GetTranslatePrefs());
   const std::string source_code = TranslateDownloadManager::GetLanguageCode(
@@ -230,8 +221,8 @@ void TranslateManager::InitiateManualTranslation(bool auto_translate,
 
   // Translate the page if it has not been translated and manual translate
   // should trigger translation automatically. Otherwise, only show the infobar.
-  if (auto_translate) {
-    TranslatePage(source_code, target_lang, triggered_from_menu);
+  if (!language_state_.IsPageTranslated() && auto_translate) {
+    TranslatePage(source_code, target_lang, false);
     return;
   }
 

components/translate/core/browser/translate_manager.h

@@ -120,8 +120,7 @@ class TranslateManager {
   // Initiate a manually triggered translation process for the current page.
   // Collect source and target languages, and show translation UI. If
   // |auto_translate| is true the page gets translated to the target language.
-  void InitiateManualTranslation(bool auto_translate = false,
-                                 bool triggered_from_menu = false);
+  void InitiateManualTranslation(bool auto_translate = false);
 
   // Returns true iff the current page could be manually translated.
   bool CanManuallyTranslate();