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.mm

 // 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.
 
 #import "chrome/browser/ui/cocoa/one_click_signin_view_controller.h"
 
 #include "base/callback_helpers.h"
 #include "base/logging.h"
 #include "base/mac/bundle_locations.h"
 #import "chrome/browser/ui/chrome_style.h"
@@ skipping 17 matching lines @@
 
 }  // namespace
 
 @interface OneClickSigninViewController ()
 - (CGFloat)initializeInformativeTextView;
 - (void)close;
 @end
 
 @implementation OneClickSigninViewController
 
+
 - (id)initWithNibName:(NSString*)nibName
           webContents:(content::WebContents*)webContents
          syncCallback:(const BrowserWindow::StartSyncCallback&)syncCallback
-        closeCallback:(const base::Closure&)closeCallback {
+        closeCallback:(const base::Closure&)closeCallback
+         isSyncDialog:(BOOL)isSyncDialog
+         errorMessage:(NSString*)errorMessage {
   if ((self = [super initWithNibName:nibName
                               bundle:base::mac::FrameworkBundle()])) {
     webContents_ = webContents;
     startSyncCallback_ = syncCallback;
     closeCallback_ = closeCallback;
-    DCHECK(!startSyncCallback_.is_null());
+    isSyncDialog_ = isSyncDialog;
+    errorMessage_ = errorMessage;
+
+    if (isSyncDialog_)
+      DCHECK(!startSyncCallback_.is_null());
   }
   return self;
 }
 
 - (void)viewWillClose {
-  if (!startSyncCallback_.is_null()) {
+  if (isSyncDialog_ && !startSyncCallback_.is_null()) {
     base::ResetAndReturn(&startSyncCallback_).Run(
         OneClickSigninSyncStarter::SYNC_WITH_DEFAULT_SETTINGS);
   }
 }
 
 - (IBAction)ok:(id)sender {
-  base::ResetAndReturn(&startSyncCallback_).Run(
+  if (isSyncDialog_) {
+    base::ResetAndReturn(&startSyncCallback_).Run(
       OneClickSigninSyncStarter::SYNC_WITH_DEFAULT_SETTINGS);
+  }
   [self close];
 }
 
 - (IBAction)onClickUndo:(id)sender {
-  base::ResetAndReturn(&startSyncCallback_).Run(
-      OneClickSigninSyncStarter::UNDO_SYNC);
+  if (isSyncDialog_) {
+    base::ResetAndReturn(&startSyncCallback_).Run(
+        OneClickSigninSyncStarter::UNDO_SYNC);
+  }
   [self close];
 }
 
 - (IBAction)onClickAdvancedLink:(id)sender {
-  base::ResetAndReturn(&startSyncCallback_).Run(
-      OneClickSigninSyncStarter::CONFIGURE_SYNC_FIRST);
+  if (isSyncDialog_) {
+    base::ResetAndReturn(&startSyncCallback_).Run(
+        OneClickSigninSyncStarter::CONFIGURE_SYNC_FIRST);
+  } else {
+    content::OpenURLParams params(GURL(chrome::kChromeUISettingsURL),
+                                content::Referrer(), CURRENT_TAB,
+                                content::PAGE_TRANSITION_LINK, false);
+    webContents_->OpenURL(params);
+  }
   [self close];
 }
 
 - (void)awakeFromNib {
   // Lay out the text controls from the bottom up.
   CGFloat totalYOffset = 0.0;
 
   totalYOffset +=
       [GTMUILocalizerAndLayoutTweaker sizeToFitView:advancedLink_].height;
   [[advancedLink_ cell] setTextColor:
       gfx::SkColorToCalibratedNSColor(chrome_style::GetLinkColor())];
 
   if (informativePlaceholderTextField_) {
     ShiftOriginY(informativePlaceholderTextField_, totalYOffset);
     totalYOffset += [self initializeInformativeTextView];
   }
 
   ShiftOriginY(messageTextField_, totalYOffset);
   totalYOffset +=
       [GTMUILocalizerAndLayoutTweaker
           sizeToFitFixedWidthTextField:messageTextField_];
 
   if (closeButton_)
     ShiftOriginY(closeButton_, totalYOffset);
 
   NSSize delta = NSMakeSize(0.0, totalYOffset);
 
+  if (!isSyncDialog_ && [errorMessage_ length] != 0)
+    [messageTextField_ setStringValue:errorMessage_];
+
   // Resize bubble and window to hold the controls.
   [GTMUILocalizerAndLayoutTweaker
       resizeViewWithoutAutoResizingSubViews:[self view]
                                       delta:delta];
 }
 
 - (CGFloat)initializeInformativeTextView {
   NSRect oldFrame = [informativePlaceholderTextField_ frame];
 
   // Replace the placeholder NSTextField with the real label NSTextView. The
   // former doesn't show links in a nice way, but the latter can't be added in
   // a xib without a containing scroll view, so create the NSTextView
   // programmatically.
   informativeTextView_.reset(
       [[HyperlinkTextView alloc] initWithFrame:oldFrame]);
   [informativeTextView_.get() setAutoresizingMask:
       [informativePlaceholderTextField_ autoresizingMask]];
   [informativeTextView_.get() setDelegate:self];
 
   // Set the text.
   NSString* learnMoreText = l10n_util::GetNSStringWithFixup(IDS_LEARN_MORE);
-  NSString* messageText =
-      l10n_util::GetNSStringWithFixup(IDS_ONE_CLICK_SIGNIN_DIALOG_MESSAGE);
-  messageText = [messageText stringByAppendingString:@" "];
+  NSString* messageText;
+
+  ui::ResourceBundle::FontStyle fontStyle = isSyncDialog_ ?
+      chrome_style::kTextFontStyle : ui::ResourceBundle::SmallFont;
   NSFont* font = ui::ResourceBundle::GetSharedInstance().GetFont(
-      chrome_style::kTextFontStyle).GetNativeFont();
+      fontStyle).GetNativeFont();
+
+  // The non-modal bubble already has a text content and only needs the
+  // Learn More link (in a smaller font).
+  if (isSyncDialog_) {
+    messageText = l10n_util::GetNSStringWithFixup(
+      IDS_ONE_CLICK_SIGNIN_DIALOG_MESSAGE);
+    messageText = [messageText stringByAppendingString:@" "];

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

What the reason for appending a space here?

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

In the modal dialog, the Learn More link is appended inline with the dialog
text. No space means it would be squished together after the period.

On 2013/04/22 18:16:51, Alexei Svitkine wrote:

+  } else {
+    messageText = @"";
+  }
+
   NSColor* linkColor =
       gfx::SkColorToCalibratedNSColor(chrome_style::GetLinkColor());
   [informativeTextView_ setMessageAndLink:messageText
                                  withLink:learnMoreText
                                  atOffset:[messageText length]
                                      font:font
                              messageColor:[NSColor blackColor]
                                 linkColor:linkColor];
 
   // Size to fit.
   [[informativePlaceholderTextField_ cell] setAttributedStringValue:
       [informativeTextView_ attributedString]];
   [GTMUILocalizerAndLayoutTweaker
         sizeToFitFixedWidthTextField:informativePlaceholderTextField_];
   NSRect newFrame = [informativePlaceholderTextField_ frame];
   [informativeTextView_ setFrame:newFrame];
 
   // Swap placeholder.
   [[informativePlaceholderTextField_ superview]
      replaceSubview:informativePlaceholderTextField_
                with:informativeTextView_.get()];
   informativePlaceholderTextField_ = nil;  // Now released.
 
   return NSHeight(newFrame) - NSHeight(oldFrame);
 }
 
 - (BOOL)textView:(NSTextView*)textView
    clickedOnLink:(id)link
          atIndex:(NSUInteger)charIndex {
+  WindowOpenDisposition location = isSyncDialog_ ?
+                                   NEW_WINDOW : NEW_FOREGROUND_TAB;
+
   content::OpenURLParams params(GURL(chrome::kChromeSyncLearnMoreURL),
-                                content::Referrer(), NEW_WINDOW,
+                                content::Referrer(), location,
                                 content::PAGE_TRANSITION_LINK, false);
   webContents_->OpenURL(params);
   return YES;
 }
 
 - (void)close {
   base::ResetAndReturn(&closeCallback_).Run();
 }
 
 @end