Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(706)

Unified Diff: chrome/browser/translate/translate_manager.cc

Issue 159883002: Move TranslatePrefs to the Translate component (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@acceptLanguages
Patch Set: Fix typo Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/translate/translate_manager.cc
diff --git a/chrome/browser/translate/translate_manager.cc b/chrome/browser/translate/translate_manager.cc
index 489a043748e184045a3d7badee0f4e328194d826..9e194be58fc97d6adc0ea472a3bee35bfc1e4dbb 100644
--- a/chrome/browser/translate/translate_manager.cc
+++ b/chrome/browser/translate/translate_manager.cc
@@ -17,7 +17,6 @@
#include "chrome/browser/chrome_notification_types.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/tab_contents/tab_util.h"
-#include "chrome/browser/translate/translate_prefs.h"
#include "chrome/browser/translate/translate_tab_helper.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/browser/ui/browser_finder.h"
@@ -32,6 +31,7 @@
#include "components/translate/core/browser/translate_download_manager.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"
@@ -299,17 +299,21 @@ void TranslateManager::InitiateTranslation(WebContents* web_contents,
return;
}
- TranslatePrefs translate_prefs(prefs);
+ scoped_ptr<TranslatePrefs> translate_prefs(
+ TranslateTabHelper::CreateTranslatePrefs(profile->GetPrefs()));
+ bool is_accept_language =
+ TranslateTabHelper::IsAcceptLanguage(profile, language_code);
// Don't translate any user black-listed languages.
- if (!TranslatePrefs::CanTranslateLanguage(profile, language_code)) {
+ if (!translate_prefs->CanTranslateLanguage(is_accept_language,
+ language_code)) {
TranslateBrowserMetrics::ReportInitiationStatus(
TranslateBrowserMetrics::INITIATION_STATUS_DISABLED_BY_CONFIG);
return;
}
// Don't translate any user black-listed URLs.
- if (translate_prefs.IsSiteBlacklisted(page_url.HostNoBrackets())) {
+ if (translate_prefs->IsSiteBlacklisted(page_url.HostNoBrackets())) {
TranslateBrowserMetrics::ReportInitiationStatus(
TranslateBrowserMetrics::INITIATION_STATUS_DISABLED_BY_CONFIG);
return;
@@ -611,8 +615,10 @@ std::string TranslateManager::GetAutoTargetLanguage(
const std::string& original_language,
PrefService* prefs) {
std::string auto_target_lang;
- if (TranslatePrefs::ShouldAutoTranslate(prefs, original_language,
- &auto_target_lang)) {
+ scoped_ptr<TranslatePrefs> translate_prefs(
+ TranslateTabHelper::CreateTranslatePrefs(prefs));
+ if (translate_prefs->ShouldAutoTranslate(original_language,
+ &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 =

Powered by Google App Engine
This is Rietveld 408576698