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

chrome/browser/ui/gtk/one_click_signin_bubble_gtk.h

Index: chrome/browser/ui/gtk/one_click_signin_bubble_gtk.h
diff --git a/chrome/browser/ui/gtk/one_click_signin_bubble_gtk.h b/chrome/browser/ui/gtk/one_click_signin_bubble_gtk.h
index 12fed1019b235c548c0b64503a745cd20b782b25..e6caa0b869196322130e162e106058f2a0d59c8e 100644
--- a/chrome/browser/ui/gtk/one_click_signin_bubble_gtk.h
+++ b/chrome/browser/ui/gtk/one_click_signin_bubble_gtk.h
@@ -9,6 +9,8 @@
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/compiler_specific.h"
+#include "chrome/browser/ui/base_window.h"
+#include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/gtk/bubble/bubble_gtk.h"
 
 typedef struct _GtkWidget GtkWidget;
@@ -16,34 +18,34 @@ typedef struct _GtkWindow GtkWindow;
 
 class BrowserWindowGtk;
 
-// Displays the one-click signin confirmation bubble (after syncing
+// Displays the one-click signin confirmation bubble (before syncing
 // has started).
 class OneClickSigninBubbleGtk : public BubbleDelegateGtk {
  public:
-  // Deletes self on close.  The given callbacks will be called if the
-  // user clicks the corresponding link.
-  OneClickSigninBubbleGtk(BrowserWindowGtk* browser_window_gtk,
-                          const base::Closure& learn_more_callback,
-                          const base::Closure& advanced_callback);
+  // Deletes self on close.  The given callback will be called if the
+  // user decides to start sync.
+  OneClickSigninBubbleGtk(
+      BrowserWindowGtk* browser_window_gtk,
+      const BrowserWindow::StartSyncCallback& start_sync_callback);
 
   // BubbleDelegateGtk implementation.
   virtual void BubbleClosing(
       BubbleGtk* bubble, bool closed_by_escape) OVERRIDE;
 
   void ClickOKForTest();
-  void ClickLearnMoreForTest();
+  void ClickUndoForTest();
   void ClickAdvancedForTest();
+  void CloseForTest();
 
  private:
   virtual ~OneClickSigninBubbleGtk();
 
-  CHROMEGTK_CALLBACK_0(OneClickSigninBubbleGtk, void, OnClickLearnMoreLink);
   CHROMEGTK_CALLBACK_0(OneClickSigninBubbleGtk, void, OnClickAdvancedLink);
   CHROMEGTK_CALLBACK_0(OneClickSigninBubbleGtk, void, OnClickOK);
+  CHROMEGTK_CALLBACK_0(OneClickSigninBubbleGtk, void, OnClickUndo);
 
   BubbleGtk* bubble_;
-  const base::Closure learn_more_callback_;
-  const base::Closure advanced_callback_;
+  BrowserWindow::StartSyncCallback start_sync_callback_;

[akalin, 2012/05/24 18:42:04]

Add a comment saying that this callback is nulled out when it's called or if the
user decides to undo, and it's called on close if it's not nulled out

[Roger Tawa OOO till Jul 10th, 2012/05/24 19:08:33]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(OneClickSigninBubbleGtk);
 };