From 80c50a920a1053bb1338ce593127b493da4276fe Mon Sep 17 00:00:00 2001 From: "jcivelli@chromium.org" Date: Wed, 16 Jun 2010 06:28:10 +0000 Subject: Hide the translate infobar when a language/site gets blacklisted. BUG=39222 TEST=See bug Review URL: http://codereview.chromium.org/2822001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49902 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/translate/translate_infobar_delegate2.cc | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/chrome/browser/translate/translate_infobar_delegate2.cc b/chrome/browser/translate/translate_infobar_delegate2.cc index 9b5395b..5598178 100644 --- a/chrome/browser/translate/translate_infobar_delegate2.cc +++ b/chrome/browser/translate/translate_infobar_delegate2.cc @@ -167,10 +167,12 @@ bool TranslateInfoBarDelegate2::IsLanguageBlacklisted() { void TranslateInfoBarDelegate2::ToggleLanguageBlacklist() { const std::string& original_lang = GetLanguageCodeAt(original_language_index()); - if (prefs_.IsLanguageBlacklisted(original_lang)) + if (prefs_.IsLanguageBlacklisted(original_lang)) { prefs_.RemoveLanguageFromBlacklist(original_lang); - else + } else { prefs_.BlacklistLanguage(original_lang); + tab_contents_->RemoveInfoBar(this); + } } bool TranslateInfoBarDelegate2::IsSiteBlacklisted() { @@ -183,10 +185,12 @@ void TranslateInfoBarDelegate2::ToggleSiteBlacklist() { if (host.empty()) return; - if (prefs_.IsSiteBlacklisted(host)) + if (prefs_.IsSiteBlacklisted(host)) { prefs_.RemoveSiteFromBlacklist(host); - else + } else { prefs_.BlacklistSite(host); + tab_contents_->RemoveInfoBar(this); + } } bool TranslateInfoBarDelegate2::ShouldAlwaysTranslate() { -- cgit v1.1