Autofill: Replace "save credit card" infobar with a bubble (Views only).

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 b377723a4d64556da41929752ca1cd52cc59ab60..784bc7558c659f54ca8be36c57588dfcc0dc4a88 100644
--- a/chrome/browser/ui/browser_window.h
+++ b/chrome/browser/ui/browser_window.h
@@ -37,6 +37,11 @@ class ToolbarActionsBar;
 
 struct WebApplicationInfo;
 
+namespace autofill {
+class SaveCardBubbleController;
+class SaveCardBubbleView;
+}
+
 namespace content {
 class WebContents;
 struct NativeWebKeyboardEvent;
@@ -245,6 +250,11 @@ class BrowserWindow : public ui::BaseWindow {
       const WebApplicationInfo& web_app_info,
       const ShowBookmarkAppBubbleCallback& callback) = 0;
 
+  // Shows the "Save credit card" bubble.
+  virtual autofill::SaveCardBubbleView* ShowSaveCreditCardBubble(

[Evan Stade, 2015/10/16 01:20:44]

need to add this to cocoa (and maybe a test/mock class or two)

[hcarmona, 2015/10/16 21:16:35]

This is only called by SaveCardBubbleControllerImpl::ShowBubble()
Will cocoa use this the same way? Maybe this function should move to the
SaveCardBubbleControllerImpl?

[bondd, 2015/10/22 02:15:17]

Yes, the plan is for Cocoa to use it the same way. AFAIK it needs to be in
BrowserWindow so that subclass BrowserView can call
GetToolbarView()->GetSaveCreditCardBubbleAnchor() to get the anchor to pass in
to the SaveCardBubbleViews constructor.

[bondd, 2015/10/22 02:15:17]

Done.

+      content::WebContents* contents,
+      autofill::SaveCardBubbleController* controller) = 0;
+
   // Shows the translate bubble.
   //
   // |is_user_gesture| is true when the bubble is shown on the user's deliberate