Translate: New Bubble UX (for the view toolkit)

chrome/browser/ui/browser.h

Index: chrome/browser/ui/browser.h
diff --git a/chrome/browser/ui/browser.h b/chrome/browser/ui/browser.h
index 0b9c93f2f13251c6694028cdb9011df2e98d4d87..9323918c698371e6573b09fd0ef9a8a7a259c97c 100644
--- a/chrome/browser/ui/browser.h
+++ b/chrome/browser/ui/browser.h
@@ -20,6 +20,7 @@
 #include "base/strings/string16.h"
 #include "chrome/browser/devtools/devtools_toggle_action.h"
 #include "chrome/browser/sessions/session_id.h"
+#include "chrome/browser/tab_contents/language_state.h"
 #include "chrome/browser/ui/bookmarks/bookmark_bar.h"
 #include "chrome/browser/ui/bookmarks/bookmark_tab_helper_delegate.h"
 #include "chrome/browser/ui/browser_navigator.h"
@@ -99,6 +100,7 @@ class Browser : public TabStripModelObserver,
                 public ChromeWebModalDialogManagerDelegate,
                 public BookmarkTabHelperDelegate,
                 public ZoomObserver,
+                public LanguageState::Observer,

[sky, 2013/10/18 14:31:10]

Having Browser implement tons of observers doesn't really scale. Move this into
BrowserLanguageStateObserver and have Browser own BrowserLanguageStateObserver.

[hajimehoshi, 2013/10/21 10:29:12]

Done.

                 public content::PageNavigator,
                 public content::NotificationObserver,
                 public ui::SelectFileDialog::Listener {
@@ -682,6 +684,10 @@ class Browser : public TabStripModelObserver,
   virtual void OnZoomChanged(content::WebContents* source,
                              bool can_show_bubble) OVERRIDE;
 
+  // Overridden from LanguageState::Observer
+  virtual void OnTranslateEnabledChanged(content::WebContents* source)
+      OVERRIDE;
+
   // Overridden from SelectFileDialog::Listener:
   virtual void FileSelected(const base::FilePath& path,
                             int index,