Allow manual translation on pages with page level notranslate meta tag

Bug: 1151492
Change-Id: I8e205b8eb39fb7ce54cec9373834980344252f92
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2556658Reviewed-by: Megan Jablonski <megjablon@chromium.org>
Commit-Queue: Anthony Cui <cuianthony@chromium.org>
Cr-Commit-Position: refs/heads/master@{#830595}

components/translate/core/browser/translate_manager.cc

@@ -242,8 +242,7 @@ bool TranslateManager::CanManuallyTranslate(bool menuLogging) {
   }
 
   if (!translate_client_->IsTranslatableURL(
-          translate_driver_->GetVisibleURL()) ||
-      !language_state_.page_needs_translation()) {
+          translate_driver_->GetVisibleURL())) {
     if (!menuLogging)
       return false;
     TranslateBrowserMetrics::ReportMenuTranslationUnavailableReason(

components/translate/core/browser/translate_manager_unittest.cc

@@ -1003,8 +1003,10 @@ TEST_F(TranslateManagerTest, CanManuallyTranslate_PageNeedsTranslation) {
   network_notifier_.SimulateOnline();
 
   translate_manager_->GetLanguageState()->LanguageDetermined("de", false);
-  EXPECT_FALSE(translate_manager_->CanManuallyTranslate());
-  EXPECT_FALSE(translate_manager_->CanManuallyTranslate(true));
+  // Users should be able to manually translate the page, even when
+  // |page_needs_translation| is false.
+  EXPECT_TRUE(translate_manager_->CanManuallyTranslate());
+  EXPECT_TRUE(translate_manager_->CanManuallyTranslate(true));
 
   translate_manager_->GetLanguageState()->LanguageDetermined("de", true);
   EXPECT_TRUE(translate_manager_->CanManuallyTranslate());