Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(876)

Unified Diff: chrome/browser/ui/cocoa/one_click_signin_bubble_controller.h

Issue 10332185: Update behavior of one-click infobar to remove modal dialog, add "undo". (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Minor comment fixes Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..54947435c8710f0567ae018dd0eeb9230584318b 100644
--- a/chrome/browser/ui/cocoa/one_click_signin_bubble_controller.h
+++ b/chrome/browser/ui/cocoa/one_click_signin_bubble_controller.h
@@ -18,11 +18,12 @@
@interface OneClickSigninBubbleController : BaseBubbleController {
@private
IBOutlet NSTextField* messageField_;
+ // TODO(akalin): learnMoreLink_ needs to be removed, but it can't until
+ // the nib is changed too.
IBOutlet NSButton* learnMoreLink_;
IBOutlet NSButton* advancedLink_;
- base::Closure learnMoreCallback_;
- base::Closure advancedCallback_;
+ base::Callback<void(bool)> start_sync_callback_;
}
// Initializes with a browser window controller, under whose wrench
@@ -32,14 +33,15 @@
// 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 base::Callback<void(bool)>&)start_sync_callback;
-// Just closes the bubble.
+// Starts sync and closes the bubble.
- (IBAction)ok:(id)sender;
-// Calls |learnMoreCallback_|.
-- (IBAction)onClickLearnMoreLink:(id)sender;
+// Just closes the bubble.
+// TODO(akalin): The following line should be uncommented once the nib is
akalin 2012/05/18 17:57:03 I think you should just remove this completely, an
Roger Tawa OOO till Jul 10th 2012/05/23 17:13:02 Done.
+// also changed.
+// - (IBAction)undo:(id)sender;
// Calls |advancedCallback_|.
- (IBAction)onClickAdvancedLink:(id)sender;

Powered by Google App Engine
This is Rietveld 408576698