Chromium Code Reviews| 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; |