Move reload processing from TranslateManager to TranslateTabHelper.

chrome/browser/translate/translate_manager.cc

 // Copyright (c) 2012 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/translate/translate_manager.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/string_split.h"
 #include "base/strings/stringprintf.h"
 #include "base/time/time.h"
-#include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/translate/translate_tab_helper.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/common/url_constants.h"
 #include "components/translate/content/common/translate_messages.h"
 #include "components/translate/core/browser/language_state.h"
 #include "components/translate/core/browser/page_translated_details.h"
 #include "components/translate/core/browser/translate_accept_languages.h"
 #include "components/translate/core/browser/translate_browser_metrics.h"
 #include "components/translate/core/browser/translate_client.h"
 #include "components/translate/core/browser/translate_download_manager.h"
 #include "components/translate/core/browser/translate_driver.h"
 #include "components/translate/core/browser/translate_error_details.h"
 #include "components/translate/core/browser/translate_language_list.h"
 #include "components/translate/core/browser/translate_prefs.h"
 #include "components/translate/core/browser/translate_script.h"
 #include "components/translate/core/browser/translate_url_util.h"
 #include "components/translate/core/common/language_detection_details.h"
 #include "components/translate/core/common/translate_constants.h"
 #include "components/translate/core/common/translate_pref_names.h"
 #include "components/translate/core/common/translate_switches.h"
 #include "content/public/browser/navigation_controller.h"
-#include "content/public/browser/navigation_details.h"
 #include "content/public/browser/navigation_entry.h"
-#include "content/public/browser/notification_details.h"
-#include "content/public/browser/notification_service.h"
-#include "content/public/browser/notification_source.h"
-#include "content/public/browser/notification_types.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
 #include "net/base/url_util.h"
 #include "net/http/http_status_code.h"
 
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/file_manager/app_id.h"
 #include "extensions/common/constants.h"
 #endif
@@ skipping 10 matching lines @@
 const char kReportLanguageDetectionErrorURL[] =
     "https://translate.google.com/translate_error?client=cr&action=langidc";
 
 // Used in kReportLanguageDetectionErrorURL to specify the original page
 // language.
 const char kSourceLanguageQueryName[] = "sl";
 
 // Used in kReportLanguageDetectionErrorURL to specify the page URL.
 const char kUrlQueryName[] = "u";
 
-// The maximum number of attempts we'll do to see if the page has finshed
-// loading before giving up the translation
-const int kMaxTranslateLoadCheckAttempts = 20;
-
 // Notifies |g_callback_list_| of translate errors.
 void NotifyTranslateError(const TranslateErrorDetails& details) {
   if (!g_callback_list_)
     return;
 
   g_callback_list_->Notify(details);
 }
 
 }  // namespace
 
@@ skipping 11 matching lines @@
   return !url.is_empty() &&
          !url.SchemeIs(content::kChromeUIScheme) &&
          !url.SchemeIs(content::kChromeDevToolsScheme) &&
 #if defined(OS_CHROMEOS)
          !(url.SchemeIs(extensions::kExtensionScheme) &&
            url.DomainIs(file_manager::kFileManagerAppId)) &&
 #endif
          !url.SchemeIs(content::kFtpScheme);
 }
 
-void TranslateManager::Observe(int type,
-                               const content::NotificationSource& source,
-                               const content::NotificationDetails& details) {
-  switch (type) {
-    case content::NOTIFICATION_NAV_ENTRY_COMMITTED: {
-      NavigationController* controller =
-          content::Source<NavigationController>(source).ptr();
-      DCHECK_EQ(&translate_tab_helper_->GetWebContents()->GetController(),
-                controller);
-      content::LoadCommittedDetails* load_details =
-          content::Details<content::LoadCommittedDetails>(details).ptr();
-      NavigationEntry* entry = controller->GetActiveEntry();
-      if (!entry) {
-        NOTREACHED();
-        return;
-      }
-
-      if (!translate_tab_helper_->GetWebContents())
-        return;
-
-      // If the navigation happened while offline don't show the translate
-      // bar since there will be nothing to translate.
-      if (load_details->http_status_code == 0 ||
-          load_details->http_status_code == net::HTTP_INTERNAL_SERVER_ERROR) {
-        return;
-      }
-
-      if (!load_details->is_main_frame &&
-          translate_driver_->GetLanguageState().translation_declined()) {
-        // Some sites (such as Google map) may trigger sub-frame navigations
-        // when the user interacts with the page.  We don't want to show a new
-        // infobar if the user already dismissed one in that case.
-        return;
-      }
-      if (entry->GetTransitionType() != content::PAGE_TRANSITION_RELOAD &&
-          load_details->type != content::NAVIGATION_TYPE_SAME_PAGE) {
-        return;
-      }
-
-      // When doing a page reload, TAB_LANGUAGE_DETERMINED is not sent,
-      // so the translation needs to be explicitly initiated, but only when the
-      // page needs translation.
-      if (!translate_driver_->GetLanguageState().page_needs_translation())
-        return;
-      // Note that we delay it as the TranslateManager gets this notification
-      // before the WebContents and the WebContents processing might remove the
-      // current infobars.  Since InitTranslation might add an infobar, it must
-      // be done after that.
-      base::MessageLoop::current()->PostTask(
-          FROM_HERE,
-          base::Bind(
-              &TranslateManager::InitiateTranslationPosted,
-              weak_method_factory_.GetWeakPtr(),
-              translate_driver_->GetLanguageState().original_language(),
-              0));
-      break;
-    }
-    default:
-      NOTREACHED();
-  }
-}
-
 // static
 scoped_ptr<TranslateManager::TranslateErrorCallbackList::Subscription>
 TranslateManager::RegisterTranslateErrorCallback(
     const TranslateManager::TranslateErrorCallback& callback) {
   if (!g_callback_list_)
     g_callback_list_ = new TranslateErrorCallbackList;
   return g_callback_list_->Add(callback);
 }
 
 TranslateManager::TranslateManager(
     TranslateTabHelper* helper,
     const std::string& accept_languages_pref_name)
-    : max_reload_check_attempts_(kMaxTranslateLoadCheckAttempts),
-      accept_languages_pref_name_(accept_languages_pref_name),
+    : accept_languages_pref_name_(accept_languages_pref_name),
       translate_tab_helper_(helper),
       translate_client_(helper),
       translate_driver_(translate_client_->GetTranslateDriver()),
-      weak_method_factory_(this) {
-
-  WebContents* web_contents = translate_tab_helper_->GetWebContents();
-
-  notification_registrar_.Add(
-      this, content::NOTIFICATION_NAV_ENTRY_COMMITTED,
-      content::Source<NavigationController>(&web_contents->GetController()));
-}
+      weak_method_factory_(this) {}
 
 void TranslateManager::InitiateTranslation(const std::string& page_lang) {
   // Short-circuit out if not in a state where initiating translation makes
   // sense.
   LanguageState& language_state = translate_driver_->GetLanguageState();
   if (!language_state.page_needs_translation() ||
       language_state.translation_pending() ||
       language_state.translation_declined() ||
       language_state.IsPageTranslated()) {
     return;
@@ skipping 110 matching lines @@
       TranslateBrowserMetrics::INITIATION_STATUS_SHOW_INFOBAR);
 
   // Prompts the user if he/she wants the page translated.
   translate_tab_helper_->ShowTranslateUI(TranslateTabHelper::BEFORE_TRANSLATE,
                                          language_code,
                                          target_lang,
                                          TranslateErrors::NONE,
                                          false);
 }
 
-void TranslateManager::InitiateTranslationPosted(const std::string& page_lang,
-                                                 int attempt) {
-  WebContents* web_contents = translate_tab_helper_->GetWebContents();
-  DCHECK(web_contents);
-
-  if (translate_driver_->GetLanguageState().translation_pending())
-    return;
-
-  // During a reload we need web content to be available before the
-  // translate script is executed. Otherwise we will run the translate script on
-  // an empty DOM which will fail. Therefore we wait a bit to see if the page
-  // has finished.
-  if ((web_contents->IsLoading()) && attempt < kMaxTranslateLoadCheckAttempts) {
-    int backoff = attempt * max_reload_check_attempts_;
-    base::MessageLoop::current()->PostDelayedTask(
-        FROM_HERE, base::Bind(&TranslateManager::InitiateTranslationPosted,
-                              weak_method_factory_.GetWeakPtr(),
-                              page_lang, ++attempt),
-        base::TimeDelta::FromMilliseconds(backoff));
-    return;
-  }
-
-  InitiateTranslation(TranslateDownloadManager::GetLanguageCode(page_lang));
-}
-
 void TranslateManager::TranslatePage(const std::string& original_source_lang,
                                      const std::string& target_lang,
                                      bool triggered_from_menu) {
   WebContents* web_contents = translate_tab_helper_->GetWebContents();
   DCHECK(web_contents);
   NavigationEntry* entry = web_contents->GetController().GetActiveEntry();
   if (!entry) {
     NOTREACHED();
     return;
   }
@@ skipping 192 matching lines @@
                                            &auto_target_lang)) {
     // We need to confirm that the saved target language is still supported.
     // Also, GetLanguageCode will take care of removing country code if any.
     auto_target_lang =
         TranslateDownloadManager::GetLanguageCode(auto_target_lang);
     if (TranslateDownloadManager::IsSupportedLanguage(auto_target_lang))
       return auto_target_lang;
   }
   return std::string();
 }