Introduce TranslateService and TranslateDownloadManager

chrome/browser/browser_process_impl.cc

Index: chrome/browser/browser_process_impl.cc
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index ac6f1c42aecd9a0cf46c996b2a8e8c68a9efd79e..22b5ee1542d4250744ac75a1c46663ae681b9e6b 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -81,6 +81,7 @@
 #include "chrome/common/url_constants.h"
 #include "chrome/installer/util/google_update_constants.h"
 #include "components/policy/core/common/policy_service.h"
+#include "components/translate/core/browser/translate_download_manager.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/child_process_security_policy.h"
 #include "content/public/browser/notification_details.h"
@@ -636,6 +637,7 @@ void BrowserProcessImpl::SetApplicationLocale(const std::string& locale) {
   locale_ = locale;
   extension_l10n_util::SetProcessLocale(locale);
   chrome::ChromeContentBrowserClient::SetApplicationLocale(locale);
+  TranslateDownloadManager::GetInstance()->set_application_locale(locale);

[droger, 2014/01/28 14:16:48]

This line can be removed if we assume that the locale never changes, as the
locale is already set in TranslateService::Initialize.

[blundell, 2014/01/29 12:25:13]

I think that this line should stay -- it does no harm, and it seems conceptually
correct to me to have this line be in this function.

On 2014/01/28 14:16:48, droger wrote:

 }
 
 DownloadStatusUpdater* BrowserProcessImpl::download_status_updater() {