[Wait to be closed] [Translate] Migrate IPCs to Mojo interfaces.

chrome/browser/translate/chrome_translate_client.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/translate/chrome_translate_client.h"
 
 #include <vector>
 
 #include "base/logging.h"
 #include "base/path_service.h"
 #include "base/strings/string_split.h"
 #include "build/build_config.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/infobars/infobar_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/translate/translate_accept_languages_factory.h"
 #include "chrome/browser/translate/translate_service.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/browser_window.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/browser/ui/translate/translate_bubble_factory.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/pref_names.h"
 #include "components/prefs/pref_service.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_download_manager.h"
 #include "components/translate/core/browser/translate_infobar_delegate.h"
 #include "components/translate/core/browser/translate_manager.h"
 #include "components/translate/core/browser/translate_prefs.h"
 #include "components/translate/core/common/language_detection_details.h"
 #include "components/variations/service/variations_service.h"
 #include "content/public/browser/notification_service.h"
@@ skipping 92 matching lines @@
     if (!auto_translate_language.empty()) {
       *target = auto_translate_language;
       return;
     }
   }
 
   *target =
       translate::TranslateManager::GetTargetLanguage(translate_prefs.get());
 }
 
+// static
+void ChromeTranslateClient::BindContentTranslateDriver(
+    content::RenderFrameHost* render_frame_host,
+    translate::mojom::ContentTranslateDriverRequest request) {
+  content::WebContents* web_contents =
+      content::WebContents::FromRenderFrameHost(render_frame_host);
+  // We try to bind to the driver, but if driver is not ready for now

[Anand Mistry (off Chromium), 2016/07/12 07:17:08]

This comment is a bit hard to parse and the formatting looks a bit weird. Also,
move it down to where the translate client is retrieved. Getting WebContents*
from an RFH seems bog-standard and no comment is necessary for that.

[leonhsl(Using Gerrit), 2016/07/12 10:15:09]

Acknowledged.

[leonhsl(Using Gerrit), 2016/07/13 09:56:18]

Done.

+  // or totally not available for this render frame host,
+  // the request will be just dropped, this would cause closing the message pipe
+  // which would raise connection error to peer side.
+  // Peer side could reconnect later when needed.
+  if (!web_contents)
+    return;
+
+  ChromeTranslateClient* instance =
+      ChromeTranslateClient::FromWebContents(web_contents);
+  if (!instance)
+    return;
+
+  instance->translate_driver().BindRequest(std::move(request));
+}
+
 translate::TranslateManager* ChromeTranslateClient::GetTranslateManager() {
   return translate_manager_.get();
 }
 
 void ChromeTranslateClient::ShowTranslateUI(
     translate::TranslateStep step,
     const std::string& source_language,
     const std::string& target_language,
     translate::TranslateErrors::Type error_type,
     bool triggered_from_menu) {
@@ skipping 91 matching lines @@
 #endif  // defined(OS_ANDROID)
 }
 
 void ChromeTranslateClient::WebContentsDestroyed() {
   // Translation process can be interrupted.
   // Destroying the TranslateManager now guarantees that it never has to deal
   // with NULL WebContents.
   translate_manager_.reset();
 }
 
-// ContentTranslateDriver::Observer implementation.
+// ContentTranslateDriverImpl::Observer implementation.
 
 void ChromeTranslateClient::OnLanguageDetermined(
     const translate::LanguageDetectionDetails& details) {
   // TODO: Remove translate notifications and have the clients be
-  // ContentTranslateDriver::Observer directly instead.
+  // ContentTranslateDriverImpl::Observer directly instead.
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_TAB_LANGUAGE_DETERMINED,
       content::Source<content::WebContents>(web_contents()),
       content::Details<const translate::LanguageDetectionDetails>(&details));
 }
 
 void ChromeTranslateClient::OnPageTranslated(
     const std::string& original_lang,
     const std::string& translated_lang,
     translate::TranslateErrors::Type error_type) {
   // TODO: Remove translate notifications and have the clients be
-  // ContentTranslateDriver::Observer directly instead.
+  // ContentTranslateDriverImpl::Observer directly instead.
   DCHECK(web_contents());
   translate::PageTranslatedDetails details;
   details.source_language = original_lang;
   details.target_language = translated_lang;
   details.error_type = error_type;
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_PAGE_TRANSLATED,
       content::Source<content::WebContents>(web_contents()),
       content::Details<translate::PageTranslatedDetails>(&details));
 }
@@ skipping 29 matching lines @@
     if (GetLanguageState().InTranslateNavigation())
       return;
   }
 
   TranslateBubbleFactory::Show(browser->window(), web_contents(), step,
                                error_type);
 #else
   NOTREACHED();
 #endif
 }