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

Unified Diff: chrome/browser/ui/browser_content_translate_driver_observer.cc

Issue 133273029: Move LanguageState to the translate component (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review comment + rebase Created 6 years, 11 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/ui/browser_content_translate_driver_observer.cc
diff --git a/chrome/browser/ui/browser_language_state_observer.cc b/chrome/browser/ui/browser_content_translate_driver_observer.cc
similarity index 59%
rename from chrome/browser/ui/browser_language_state_observer.cc
rename to chrome/browser/ui/browser_content_translate_driver_observer.cc
index 5487af262a7e891be4e7b648151f9b8c011aa433..3ab1cf05c1582a49a1130b132724f7b7df2a2f6e 100644
--- a/chrome/browser/ui/browser_language_state_observer.cc
+++ b/chrome/browser/ui/browser_content_translate_driver_observer.cc
@@ -2,33 +2,34 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "chrome/browser/ui/browser_language_state_observer.h"
+#include "chrome/browser/ui/browser_content_translate_driver_observer.h"
-#include "chrome/browser/tab_contents/language_state.h"
#include "chrome/browser/translate/translate_tab_helper.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/browser/ui/browser_window.h"
#include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "components/translate/core/browser/language_state.h"
-BrowserLanguageStateObserver::BrowserLanguageStateObserver(Browser* browser)
- : browser_(browser) {
+BrowserContentTranslateDriverObserver::BrowserContentTranslateDriverObserver(
+ Browser* browser) : browser_(browser) {
}
-BrowserLanguageStateObserver::~BrowserLanguageStateObserver() {
+BrowserContentTranslateDriverObserver::
+ ~BrowserContentTranslateDriverObserver() {
}
-void BrowserLanguageStateObserver::OnIsPageTranslatedChanged(
+void BrowserContentTranslateDriverObserver::OnIsPageTranslatedChanged(
content::WebContents* source) {
if (source == browser_->tab_strip_model()->GetActiveWebContents()) {
TranslateTabHelper* translate_tab_helper =
TranslateTabHelper::FromWebContents(source);
- LanguageState& language_state = translate_tab_helper->language_state();
+ LanguageState& language_state = translate_tab_helper->GetLanguageState();
browser_->window()->SetTranslateIconToggled(
language_state.IsPageTranslated());
}
}
-void BrowserLanguageStateObserver::OnTranslateEnabledChanged(
+void BrowserContentTranslateDriverObserver::OnTranslateEnabledChanged(
content::WebContents* source) {
if (source == browser_->tab_strip_model()->GetActiveWebContents())
browser_->window()->UpdateToolbar(source);
« no previous file with comments | « chrome/browser/ui/browser_content_translate_driver_observer.h ('k') | chrome/browser/ui/browser_language_state_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698