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

Unified Diff: chrome/browser/tab_contents/language_state.cc

Issue 25373009: Translate: New Bubble UX (for the view toolkit) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Bug fix: view id on tests Created 7 years, 2 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
« no previous file with comments | « chrome/browser/tab_contents/language_state.h ('k') | chrome/browser/tab_contents/language_state_observer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/tab_contents/language_state.cc
diff --git a/chrome/browser/tab_contents/language_state.cc b/chrome/browser/tab_contents/language_state.cc
index eb298954ec86feeba2233d525eea723d77f2f644..1567ad2a1470e9bddd1e4a8ae5475d6978fc4b63 100644
--- a/chrome/browser/tab_contents/language_state.cc
+++ b/chrome/browser/tab_contents/language_state.cc
@@ -4,9 +4,11 @@
#include "chrome/browser/tab_contents/language_state.h"
+#include "chrome/browser/tab_contents/language_state_observer.h"
#include "content/public/browser/navigation_controller.h"
#include "content/public/browser/navigation_details.h"
#include "content/public/browser/navigation_entry.h"
+#include "content/public/browser/web_contents.h"
using content::NavigationController;
@@ -15,7 +17,9 @@ LanguageState::LanguageState(NavigationController* nav_controller)
page_needs_translation_(false),
translation_pending_(false),
translation_declined_(false),
- in_page_navigation_(false) {
+ in_page_navigation_(false),
+ translate_enabled_(false),
+ observer_(NULL) {
}
LanguageState::~LanguageState() {
@@ -44,6 +48,8 @@ void LanguageState::DidNavigate(
translation_pending_ = false;
translation_declined_ = false;
+
+ SetTranslateEnabled(false);
}
void LanguageState::LanguageDetermined(const std::string& page_language,
@@ -84,3 +90,19 @@ std::string LanguageState::AutoTranslateTo() const {
return std::string();
}
+
+void LanguageState::SetTranslateEnabled(bool value) {
+ if (translate_enabled_ == value)
+ return;
+
+ translate_enabled_ = value;
+ if (observer_) {
+ content::WebContents* web_contents =
+ navigation_controller_->GetWebContents();
+ observer_->OnTranslateEnabledChanged(web_contents);
+ }
+}
+
+bool LanguageState::HasLanguageChanged() const {
+ return original_lang_ != prev_original_lang_;
+}
« no previous file with comments | « chrome/browser/tab_contents/language_state.h ('k') | chrome/browser/tab_contents/language_state_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698