Translate: New Bubble UX (for the view toolkit)

chrome/browser/ui/browser_window.h

Index: chrome/browser/ui/browser_window.h
diff --git a/chrome/browser/ui/browser_window.h b/chrome/browser/ui/browser_window.h
index 7811ace7de93c4d798585a35e452ad2098100279..233e71e5c15c524e0b7f6ce98bc945ef1576cdaf 100644
--- a/chrome/browser/ui/browser_window.h
+++ b/chrome/browser/ui/browser_window.h
@@ -12,6 +12,7 @@
 #include "chrome/browser/ui/fullscreen/fullscreen_exit_bubble_type.h"
 #include "chrome/browser/ui/host_desktop.h"
 #include "chrome/browser/ui/sync/one_click_signin_sync_starter.h"
+#include "chrome/browser/ui/translate/translate_bubble_model.h"
 #include "chrome/common/content_settings_types.h"
 #include "ui/base/base_window.h"
 #include "ui/base/window_open_disposition.h"
@@ -219,6 +220,11 @@ class BrowserWindow : public ui::BaseWindow {
   // TODO(yosin): Make ShowBookmarkPrompt pure virtual.
   virtual void ShowBookmarkPrompt() {}
 
+  // Shows the translate bubble.
+  virtual void ShowTranslateBubble(

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

Why does this and GetTranslateBUbbleModel need to be on BrowserWindow? I think
I'm ok with ShowTranslateBubble, but GetTranslateBubbleModel? That shouldn't be
on browserwindow.

[hajimehoshi, 2013/10/22 10:30:25]

Removed GetTranslateBubbleModel.

+      content::WebContents* contents,
+      TranslateBubbleModel::ViewState view_state) = 0;
+
 #if defined(ENABLE_ONE_CLICK_SIGNIN)
   enum OneClickSigninBubbleType {
     ONE_CLICK_SIGNIN_BUBBLE_TYPE_BUBBLE,
@@ -402,6 +408,9 @@ class BrowserWindowTesting {
   virtual ToolbarView* GetToolbarView() const = 0;
 #endif
 
+  // Returns the Translate bubble model. This may return NULL.
+  virtual TranslateBubbleModel* GetTranslateBubbleModel() const = 0;
+
  protected:
   virtual ~BrowserWindowTesting() {}
 };