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 f219b742056d4fdb1b57dc25d8605dfce7cc7a66..bd3ffe22ac60673658d0ad3345f3083fedc1eb5e 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.h"
 #include "chrome/common/content_settings_types.h"
 #include "ui/base/base_window.h"
 #include "ui/base/window_open_disposition.h"
@@ -222,6 +223,11 @@ class BrowserWindow : public ui::BaseWindow {
   // TODO(yosin): Make ShowBookmarkPrompt pure virtual.
   virtual void ShowBookmarkPrompt() {}
 
+  // Shows the translate bubble.
+  // TODO(hajimehoshi): Make this pure virtual.

[Takashi Toyoshima, 2013/10/08 14:54:09]

Yeah, this TODO means we don't implement this method in any child class for
testing. At least, file a bug for this, and add an URL link for that here. And
of course, writing a test is better than that.

[hajimehoshi, 2013/10/10 11:07:10]

OK, I'll write the test.

+  virtual void ShowTranslateBubble(content::WebContents* contents,
+                                   TranslateBubble::Type type) {}
+
 #if defined(ENABLE_ONE_CLICK_SIGNIN)
   enum OneClickSigninBubbleType {
     ONE_CLICK_SIGNIN_BUBBLE_TYPE_BUBBLE,