Mac: Display a native bubble (instead of the JS one) after the web signin flow. (branched from http…

chrome/browser/ui/cocoa/one_click_signin_view_controller.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_COCOA_ONE_CLICK_SIGNIN_VIEW_CONTROLLER_H_
 #define CHROME_BROWSER_UI_COCOA_ONE_CLICK_SIGNIN_VIEW_CONTROLLER_H_
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/callback.h"
 #include "base/memory/scoped_nsobject.h"
 #include "chrome/browser/ui/browser_window.h"
 
 @class BrowserWindowController;
 namespace content {
 class WebContents;
 }
 @class HyperlinkTextView;
 
 // View controller for the one-click signin confirmation UI.
 @interface OneClickSigninViewController : NSViewController<NSTextViewDelegate> {
  @private
   IBOutlet NSTextField* messageTextField_;
   IBOutlet NSTextField* informativePlaceholderTextField_;
   IBOutlet NSButton* advancedLink_;
   IBOutlet NSButton* closeButton_;
 
+  BOOL isSyncDialog_;
+  NSString* errorMessage_;
+
   // Text fields don't work as well with embedded links as text views, but
   // text views cannot conveniently be created in IB. The xib file contains
   // a text field |informativePlaceholderTextField_| that's replaced by this
   // text view |promo_| in -awakeFromNib.
   scoped_nsobject<HyperlinkTextView> informativeTextView_;
   BrowserWindow::StartSyncCallback startSyncCallback_;
   base::Closure closeCallback_;
   content::WebContents* webContents_;
 }
 
+// |errorMessage| is an alternate message that will be displayed in the

[Alexei Svitkine (slow), 2013/04/22 18:16:51]

Add a sentence to describe the overall method.

+// case of an authentication error, |syncCallback| is called to start sync if
+// |isSyncDialog| is YES. |webContents| is used to open the Learn More and
+// Advanced links and |callback| is called when the view is closing

[Alexei Svitkine (slow), 2013/04/22 18:16:51]

Close the sentence - i.e. you need a period.

[noms (inactive), 2013/04/23 18:52:48]

Done.

 - (id)initWithNibName:(NSString*)nibName
           webContents:(content::WebContents*)webContents
          syncCallback:(const BrowserWindow::StartSyncCallback&)syncCallback
-        closeCallback:(const base::Closure&)callback;
+        closeCallback:(const base::Closure&)callback
+         isSyncDialog:(BOOL)isSyncDialog
+         errorMessage:(NSString*)errorMessage;
 
 // Called before the view is closed.
 - (void)viewWillClose;
 
 // Starts sync and closes the bubble.
 - (IBAction)ok:(id)sender;
 
 - (IBAction)onClickUndo:(id)sender;
 
 // Calls |advancedCallback_|.
 - (IBAction)onClickAdvancedLink:(id)sender;
 
 @end
 
 #endif  // CHROME_BROWSER_UI_COCOA_ONE_CLICK_SIGNIN_VIEW_CONTROLLER_H_