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

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

Issue 10830353: Introduce InfoBarTabService API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Respond to review comments, merge to LKGR Created 8 years, 4 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_infobar_delegate.cc
diff --git a/chrome/browser/translate/translate_infobar_delegate.cc b/chrome/browser/translate/translate_infobar_delegate.cc
index e7e0b2e2a203362c0ef137c6f1e2eeab671322d4..cd0566e1d7dbaec5a8d52105dbdf63ca029350af 100644
--- a/chrome/browser/translate/translate_infobar_delegate.cc
+++ b/chrome/browser/translate/translate_infobar_delegate.cc
@@ -114,29 +114,29 @@ void TranslateInfoBarDelegate::SetTargetLanguage(size_t language_index) {
void TranslateInfoBarDelegate::Translate() {
const std::string& original_language_code = GetOriginalLanguageCode();
- if (!owner()->web_contents()->GetBrowserContext()->IsOffTheRecord()) {
+ if (!owner()->GetWebContents()->GetBrowserContext()->IsOffTheRecord()) {
prefs_.ResetTranslationDeniedCount(original_language_code);
prefs_.IncrementTranslationAcceptedCount(original_language_code);
}
- TranslateManager::GetInstance()->TranslatePage(owner()->web_contents(),
+ TranslateManager::GetInstance()->TranslatePage(owner()->GetWebContents(),
GetLanguageCodeAt(original_language_index()),
GetLanguageCodeAt(target_language_index()));
}
void TranslateInfoBarDelegate::RevertTranslation() {
- TranslateManager::GetInstance()->RevertTranslation(owner()->web_contents());
+ TranslateManager::GetInstance()->RevertTranslation(owner()->GetWebContents());
RemoveSelf();
}
void TranslateInfoBarDelegate::ReportLanguageDetectionError() {
TranslateManager::GetInstance()->
- ReportLanguageDetectionError(owner()->web_contents());
+ ReportLanguageDetectionError(owner()->GetWebContents());
}
void TranslateInfoBarDelegate::TranslationDeclined() {
const std::string& original_language_code = GetOriginalLanguageCode();
- if (!owner()->web_contents()->GetBrowserContext()->IsOffTheRecord()) {
+ if (!owner()->GetWebContents()->GetBrowserContext()->IsOffTheRecord()) {
prefs_.ResetTranslationAcceptedCount(original_language_code);
prefs_.IncrementTranslationDeniedCount(original_language_code);
}
@@ -147,7 +147,7 @@ void TranslateInfoBarDelegate::TranslationDeclined() {
// happens when a load stops. That could happen multiple times, including
// after the user already declined the translation.)
TranslateTabHelper* helper = TabContents::FromWebContents(
- owner()->web_contents())->translate_tab_helper();
+ owner()->GetWebContents())->translate_tab_helper();
helper->language_state().set_translation_declined(true);
}
@@ -263,7 +263,7 @@ void TranslateInfoBarDelegate::MessageInfoBarButtonPressed() {
return;
}
// This is the "Try again..." case.
- TranslateManager::GetInstance()->TranslatePage(owner()->web_contents(),
+ TranslateManager::GetInstance()->TranslatePage(owner()->GetWebContents(),
GetOriginalLanguageCode(), GetTargetLanguageCode());
}
@@ -273,13 +273,13 @@ bool TranslateInfoBarDelegate::ShouldShowMessageInfoBarButton() {
bool TranslateInfoBarDelegate::ShouldShowNeverTranslateButton() {
DCHECK_EQ(BEFORE_TRANSLATE, type_);
- return !owner()->web_contents()->GetBrowserContext()->IsOffTheRecord() &&
+ return !owner()->GetWebContents()->GetBrowserContext()->IsOffTheRecord() &&
(prefs_.GetTranslationDeniedCount(GetOriginalLanguageCode()) >= 3);
}
bool TranslateInfoBarDelegate::ShouldShowAlwaysTranslateButton() {
DCHECK_EQ(BEFORE_TRANSLATE, type_);
- return !owner()->web_contents()->GetBrowserContext()->IsOffTheRecord() &&
+ return !owner()->GetWebContents()->GetBrowserContext()->IsOffTheRecord() &&
(prefs_.GetTranslationAcceptedCount(GetOriginalLanguageCode()) >= 3);
}
@@ -402,6 +402,6 @@ TranslateInfoBarDelegate*
std::string TranslateInfoBarDelegate::GetPageHost() {
NavigationEntry* entry =
- owner()->web_contents()->GetController().GetActiveEntry();
+ owner()->GetWebContents()->GetController().GetActiveEntry();
return entry ? entry->GetURL().HostNoBrackets() : std::string();
}
« no previous file with comments | « chrome/browser/translate/translate_infobar_delegate.h ('k') | chrome/browser/translate/translate_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698