Update behavior of one-click infobar to remove modal dialog, add "undo".

chrome/browser/ui/cocoa/one_click_signin_bubble_controller.h

Index: chrome/browser/ui/cocoa/one_click_signin_bubble_controller.h
diff --git a/chrome/browser/ui/cocoa/one_click_signin_bubble_controller.h b/chrome/browser/ui/cocoa/one_click_signin_bubble_controller.h
index 0a0261b36464c3ec2c452936d4c4889f1733adaf..ca1627f7d00735bf3a0381b2aafe2c082fed2aa3 100644
--- a/chrome/browser/ui/cocoa/one_click_signin_bubble_controller.h
+++ b/chrome/browser/ui/cocoa/one_click_signin_bubble_controller.h
@@ -9,6 +9,7 @@
 #import <Cocoa/Cocoa.h>
 
 #include "base/callback.h"
+#include "chrome/browser/ui/browser_window.h"
 #import "chrome/browser/ui/cocoa/base_bubble_controller.h"
 
 @class BrowserWindowController;
@@ -18,11 +19,14 @@
 @interface OneClickSigninBubbleController : BaseBubbleController {
  @private
   IBOutlet NSTextField* messageField_;
+  // TODO(akalin): learnMoreLink_ needs to be removed, but it can't until

[Peter Kasting, 2012/05/24 22:25:18]

Nit: can't -> can't be

[Roger Tawa OOO till Jul 10th, 2012/05/25 16:04:08]

Done.

+  // the nib is changed too.
   IBOutlet NSButton* learnMoreLink_;
   IBOutlet NSButton* advancedLink_;
 
-  base::Closure learnMoreCallback_;
-  base::Closure advancedCallback_;
+  // TODO(akalin): Make sure this callback is called only once, like on
+  // other platforms.
+  BrowserWindow::StartSyncCallback start_sync_callback_;
 }
 
 // Initializes with a browser window controller, under whose wrench
@@ -32,15 +36,13 @@
 // The bubble is not automatically displayed; call showWindow:id to
 // display.  The bubble is auto-released on close.
 - (id)initWithBrowserWindowController:(BrowserWindowController*)controller
-                    learnMoreCallback:(const base::Closure&)learnMoreCallback
-                     advancedCallback:(const base::Closure&)advancedCallback;
+                  start_sync_callback:
+                      (const BrowserWindow::StartSyncCallback&)
+                          start_sync_callback;
 
-// Just closes the bubble.
+// Starts sync and closes the bubble.
 - (IBAction)ok:(id)sender;
 
-// Calls |learnMoreCallback_|.
-- (IBAction)onClickLearnMoreLink:(id)sender;
-
 // Calls |advancedCallback_|.
 - (IBAction)onClickAdvancedLink:(id)sender;