[rAc OSX] Factor out AutofillHeader class to contain dialog header elements.

chrome/browser/ui/cocoa/autofill/autofill_dialog_window_controller.mm

 // Copyright 2013 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/autofill/autofill_dialog_window_controller.h"
 
 #include "base/mac/foundation_util.h"
 #include "base/mac/scoped_nsobject.h"
 #include "base/strings/sys_string_conversions.h"
 #include "chrome/browser/ui/autofill/autofill_dialog_view_delegate.h"
 #include "chrome/browser/ui/chrome_style.h"
-#import "chrome/browser/ui/cocoa/autofill/autofill_account_chooser.h"
 #include "chrome/browser/ui/cocoa/autofill/autofill_dialog_cocoa.h"
 #include "chrome/browser/ui/cocoa/autofill/autofill_dialog_constants.h"
+#import "chrome/browser/ui/cocoa/autofill/autofill_header.h"
 #import "chrome/browser/ui/cocoa/autofill/autofill_input_field.h"
 #import "chrome/browser/ui/cocoa/autofill/autofill_loading_shield_controller.h"
 #import "chrome/browser/ui/cocoa/autofill/autofill_main_container.h"
 #import "chrome/browser/ui/cocoa/autofill/autofill_overlay_controller.h"
 #import "chrome/browser/ui/cocoa/autofill/autofill_section_container.h"
 #import "chrome/browser/ui/cocoa/autofill/autofill_sign_in_container.h"
 #import "chrome/browser/ui/cocoa/autofill/autofill_textfield.h"
 #import "chrome/browser/ui/cocoa/constrained_window/constrained_window_custom_window.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_view.h"
 #include "grit/generated_resources.h"
 #import "ui/base/cocoa/flipped_view.h"
 #include "ui/base/cocoa/window_size_constants.h"
 #include "ui/base/l10n/l10n_util.h"
 
 namespace {
-const CGFloat kAccountChooserHeight = 20.0;
+
 const CGFloat kMinimumContentsHeight = 101;
 
-// Height of all decorations & paddings on main dialog together.
-const CGFloat kDecorationHeight = kAccountChooserHeight +
-                                  autofill::kDetailVerticalPadding +
-                                  chrome_style::kClientBottomPadding +
-                                  chrome_style::kTitleTopPadding;
 }  // namespace
 
 #pragma mark Field Editor
 
 @interface AutofillDialogFieldEditor : NSTextView
 @end
 
 
 @implementation AutofillDialogFieldEditor
 
@@ skipping 39 matching lines @@
   }
   return fieldEditor_.get();
 }
 
 @end
 
 
 @implementation AutofillDialogWindowController
 
 - (id)initWithWebContents:(content::WebContents*)webContents
-      autofillDialog:(autofill::AutofillDialogCocoa*)autofillDialog {
+                   dialog:(autofill::AutofillDialogCocoa*)dialog {
   DCHECK(webContents);
 
   base::scoped_nsobject<ConstrainedWindowCustomWindow> window(
       [[ConstrainedWindowCustomWindow alloc]
           initWithContentRect:ui::kWindowSizeDeterminedLater]);
 
   if ((self = [super initWithWindow:window])) {
     [window setDelegate:self];
     webContents_ = webContents;
-    autofillDialog_ = autofillDialog;
+    dialog_ = dialog;
+
+    header_.reset([[AutofillHeader alloc] initWithDelegate:dialog->delegate()]);
 
     mainContainer_.reset([[AutofillMainContainer alloc]
-                             initWithDelegate:autofillDialog->delegate()]);
+                             initWithDelegate:dialog->delegate()]);
     [mainContainer_ setTarget:self];
 
     signInContainer_.reset(
-        [[AutofillSignInContainer alloc] initWithDialog:autofillDialog]);
+        [[AutofillSignInContainer alloc] initWithDialog:dialog]);
     [[signInContainer_ view] setHidden:YES];
 
-    // Set dialog title.
-    titleTextField_.reset([[NSTextField alloc] initWithFrame:NSZeroRect]);
-    [titleTextField_ setEditable:NO];
-    [titleTextField_ setBordered:NO];
-    [titleTextField_ setDrawsBackground:NO];
-    [titleTextField_ setFont:[NSFont systemFontOfSize:15.0]];
-    [titleTextField_ setStringValue:
-        base::SysUTF16ToNSString(autofillDialog->delegate()->DialogTitle())];
-    [titleTextField_ sizeToFit];
-
-    accountChooser_.reset([[AutofillAccountChooser alloc]
-                              initWithFrame:NSZeroRect
-                                   delegate:autofillDialog->delegate()]);
-
     loadingShieldController_.reset(
-        [[AutofillLoadingShieldController alloc]
-            initWithDelegate:autofillDialog->delegate()]);
+        [[AutofillLoadingShieldController alloc] initWithDelegate:
+            dialog->delegate()]);
     [[loadingShieldController_ view] setHidden:YES];
 
     overlayController_.reset(
         [[AutofillOverlayController alloc] initWithDelegate:
-            autofillDialog->delegate()]);
+            dialog->delegate()]);
     [[overlayController_ view] setHidden:YES];
 
     // This needs a flipped content view because otherwise the size
     // animation looks odd. However, replacing the contentView for constrained
     // windows does not work - it does custom rendering.
     base::scoped_nsobject<NSView> flippedContentView(
         [[FlippedView alloc] initWithFrame:
             [[[self window] contentView] frame]]);
     [flippedContentView setSubviews:
-        @[accountChooser_,
-          titleTextField_,
+        @[header_,
           [mainContainer_ view],
           [signInContainer_ view],
           [loadingShieldController_ view],
           [overlayController_ view]]];
     [flippedContentView setAutoresizingMask:
         (NSViewWidthSizable | NSViewHeightSizable)];
     [[[self window] contentView] addSubview:flippedContentView];
-    [mainContainer_ setAnchorView:[[accountChooser_ subviews] objectAtIndex:1]];
+    [mainContainer_ setAnchorView:[header_ anchorView]];
   }
   return self;
 }
 
 - (void)dealloc {
   [[NSNotificationCenter defaultCenter] removeObserver:self];
   [super dealloc];
 }
 
 - (CGFloat)maxHeight {
   NSRect dialogFrameRect = [[self window] frame];
   NSRect browserFrameRect =
       [webContents_->GetView()->GetTopLevelNativeWindow() frame];
   dialogFrameRect.size.height =
       NSMaxY(dialogFrameRect) - NSMinY(browserFrameRect);
   dialogFrameRect = [[self window] contentRectForFrameRect:dialogFrameRect];
   return NSHeight(dialogFrameRect);
 }
 
 - (void)updateSignInSizeConstraints {
   // Adjust for the size of all decorations and paddings outside main content.
-  CGFloat minHeight = kMinimumContentsHeight - kDecorationHeight;
-  CGFloat maxHeight = std::max([self maxHeight] - kDecorationHeight, minHeight);
+  CGFloat decorationHeight =
+      [header_ frame].size.height + chrome_style::kClientBottomPadding;
+  CGFloat minHeight = kMinimumContentsHeight - decorationHeight;
+  CGFloat maxHeight = std::max([self maxHeight] - decorationHeight, minHeight);
   CGFloat width = NSWidth([[[self window] contentView] frame]);
 
   [signInContainer_ constrainSizeToMinimum:NSMakeSize(width, minHeight)
                                    maximum:NSMakeSize(width, maxHeight)];
 }
 
 - (void)onContentViewFrameDidChange:(NSNotification*)notification {
   [self updateSignInSizeConstraints];
   if ([[signInContainer_ view] isHidden])
     [self requestRelayout];
@@ skipping 12 matching lines @@
 
 - (NSSize)preferredSize {
   NSSize size;
 
   // Overall size is determined by either main container or sign in view.
   if ([[signInContainer_ view] isHidden])
     size = [mainContainer_ preferredSize];
   else
     size = [signInContainer_ preferredSize];
 
   // Always make room for the header.

[groby-ooo-7-16, 2013/11/22 14:51:38]

This comment is confusing if combined with the bottom padding

[Ilya Sherman, 2013/11/23 00:23:54]

Done.

-  size.height += kDecorationHeight;
+  size.height +=
+      [header_ frame].size.height + chrome_style::kClientBottomPadding;

[groby-ooo-7-16, 2013/11/22 14:51:38]

You might want to use heightForWidth here. (Since preferredSize is "what size
would you _like_ to be")

[Ilya Sherman, 2013/11/23 00:23:54]

Done.

 
   if (![[overlayController_ view] isHidden]) {
     CGFloat height = [overlayController_ heightForWidth:size.width];
     if (height != 0.0)
       size.height = height;
   }
 
   // Show as much of the main view as is possible without going past the
   // bottom of the browser window.
   size.height = std::min(size.height, [self maxHeight]);
 
   return size;
 }
 
 - (void)performLayout {
   NSRect contentRect = NSZeroRect;
   contentRect.size = [self preferredSize];
   NSRect clientRect = contentRect;
-  clientRect.origin.y = chrome_style::kTitleTopPadding;
-  clientRect.size.height -= chrome_style::kTitleTopPadding +
-                            chrome_style::kClientBottomPadding;
+  clientRect.size.height -= chrome_style::kClientBottomPadding;
 
-  [titleTextField_ setStringValue:
-      base::SysUTF16ToNSString(autofillDialog_->delegate()->DialogTitle())];
-  [titleTextField_ sizeToFit];
+  CGFloat headerHeight = [header_ heightForWidth:NSWidth(clientRect)];
+  NSRect headerRect, mainRect;
+  NSDivideRect(clientRect, &headerRect, &mainRect, headerHeight, NSMinYEdge);
 
-  NSRect headerRect, mainRect, titleRect, dummyRect;
-  NSDivideRect(clientRect, &headerRect, &mainRect,
-               kAccountChooserHeight, NSMinYEdge);
-  NSDivideRect(mainRect, &dummyRect, &mainRect,
-               autofill::kDetailVerticalPadding, NSMinYEdge);
-  headerRect = NSInsetRect(headerRect, chrome_style::kHorizontalPadding, 0);
-  NSDivideRect(headerRect, &titleRect, &headerRect,
-               NSWidth([titleTextField_ frame]), NSMinXEdge);
+  [header_ setFrame:headerRect];
+  [header_ performLayout];
 
-  // Align baseline of title with bottom of accountChooser.
-  base::scoped_nsobject<NSLayoutManager> layout_manager(
-      [[NSLayoutManager alloc] init]);
-  NSFont* titleFont = [titleTextField_ font];
-  titleRect.origin.y += NSHeight(titleRect) -
-      [layout_manager defaultBaselineOffsetForFont:titleFont];
-  [titleTextField_ setFrame:titleRect];
-
-  [accountChooser_ setFrame:headerRect];
-  [accountChooser_ performLayout];
   if ([[signInContainer_ view] isHidden]) {
     [[mainContainer_ view] setFrame:mainRect];
     [mainContainer_ performLayout];
   } else {
     [[signInContainer_ view] setFrame:mainRect];
   }
 
   [[loadingShieldController_ view] setFrame:contentRect];
   [loadingShieldController_ performLayout];
 
   [[overlayController_ view] setFrame:contentRect];
   [overlayController_ performLayout];
 
   NSRect frameRect = [[self window] frameRectForContentRect:contentRect];
   [[self window] setFrame:frameRect display:YES];
   [[self window] recalculateKeyViewLoop];
 }
 
 - (IBAction)accept:(id)sender {
   if ([mainContainer_ validate])
-    autofillDialog_->delegate()->OnAccept();
+    dialog_->delegate()->OnAccept();
   else
     [mainContainer_ makeFirstInvalidInputFirstResponder];
 }
 
 - (IBAction)cancel:(id)sender {
-  autofillDialog_->delegate()->OnCancel();
-  autofillDialog_->PerformClose();
+  dialog_->delegate()->OnCancel();
+  dialog_->PerformClose();
 }
 
 - (void)show {
   // Resizing the browser causes the ConstrainedWindow to move.
   // Observe that to allow resizes based on browser size.
   // NOTE: This MUST come last after all initial setup is done, because there
   // is an immediate notification post registration.
   DCHECK([self window]);
   [[NSNotificationCenter defaultCenter]
       addObserver:self
          selector:@selector(onContentViewFrameDidChange:)
              name:NSWindowDidMoveNotification
            object:[self window]];
 
   [self updateAccountChooser];
   [self updateNotificationArea];
   [self requestRelayout];
 }
 
 - (void)hide {
-  autofillDialog_->delegate()->OnCancel();
-  autofillDialog_->PerformClose();
+  dialog_->delegate()->OnCancel();
+  dialog_->PerformClose();
 }
 
 - (void)updateNotificationArea {
   [mainContainer_ updateNotificationArea];
 }
 
 - (void)updateAccountChooser {
-  [accountChooser_ update];
+  [header_ update];
   [mainContainer_ updateLegalDocuments];
 
   // For the duration of the loading shield, hide the main contents.
   // This prevents the currently focused text field "shining through".
   // No need to remember previous state, because the loading shield
   // always flows through to the main container.
   [loadingShieldController_ update];
   [[mainContainer_ view] setHidden:![[loadingShieldController_ view] isHidden]];
 }
 
@@ skipping 88 matching lines @@
 
 - (content::WebContents*)getSignInWebContents {
   return [signInContainer_ webContents];
 }
 
 - (BOOL)isShowingOverlay {
   return ![[overlayController_ view] isHidden];
 }
 
 @end