[rAc] [OSX] Reduce gap between "shipping address" label and the shipping address.

chrome/browser/ui/cocoa/autofill/autofill_suggestion_container.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_suggestion_container.h"
 
 #include <algorithm>
 #include <cmath>
 
 #include "base/logging.h"
@@ skipping 17 matching lines @@
 // Technically, CGFLOAT_MAX should do. Practically, it runs into several issues.
 // #1) Many computations on Retina devices overflow with that value.
 // #2) In this particular use case, it results in the message
 //     "CGAffineTransformInvert: singular matrix."
 const CGFloat kInfiniteSize = 1.0e6;
 
 // A line fragment padding that creates the same visual look as text layout in
 // an NSTextField does. (Which UX feedback was based on)
 const CGFloat kLineFragmentPadding = 2.0;
 
-// Padding added on top of the label so its first line looks centered with
-// respect to the input field.
-const CGFloat kLabelTopPadding = 5.0;

[groby-ooo-7-16, 2013/12/13 07:19:39]

So, what happens with a CVV input? Is that not centered any more?

[Ilya Sherman, 2013/12/13 07:48:11]

Good call, that was very broken.  Fixed.

-
 }
 
 // An attachment cell for a single icon - takes care of proper alignment of
 // text and icon.
 @interface IconAttachmentCell : NSTextAttachmentCell {
   CGFloat baseline_;  // The cell's baseline adjustment.
 }
 
 // Adjust the cell's baseline so that the lower edge of the image aligns with
 // the longest descender, not the font baseline
@@ skipping 50 matching lines @@
 
 - (void)loadView {
   label_.reset([[NSTextView alloc] initWithFrame:NSZeroRect]);
   [[label_ textContainer] setLineFragmentPadding:kLineFragmentPadding];
   [label_ setEditable:NO];
   [label_ setSelectable:NO];
   [label_ setDrawsBackground:NO];
 
   base::scoped_nsobject<NSMutableParagraphStyle> paragraphStyle(
       [[NSMutableParagraphStyle alloc] init]);
-  [paragraphStyle setLineHeightMultiple:1.5];
+  [paragraphStyle setLineSpacing:0.5 * [[label_ font] pointSize]];

[groby-ooo-7-16, 2013/12/13 07:19:39]

Why the change from lineHeightMultiple to lineSpacing?

[Ilya Sherman, 2013/12/13 07:48:11]

Because lineHeightMultiple reserves space above the first line, whereas
lineSpacing only affects intra-line spacing.

   [label_ setDefaultParagraphStyle:paragraphStyle];
 
   inputField_.reset([[AutofillTextField alloc] initWithFrame:NSZeroRect]);
   [inputField_ setHidden:YES];
 
   spacer_.reset([[NSBox alloc] initWithFrame:NSZeroRect]);
   [spacer_ setBoxType:NSBoxSeparator];
   [spacer_ setBorderType:NSLineBorder];
 
   base::scoped_nsobject<NSView> view([[NSView alloc] initWithFrame:NSZeroRect]);
@@ skipping 41 matching lines @@
 
   // Enforce fixed width.
   NSSize frameSize = NSMakeSize(autofill::kFieldWidth,
                                 NSHeight([inputField_ frame]));
   [inputField_ setFrameSize:frameSize];
 }
 
 
 - (NSSize)preferredSize {
   NSSize size = [label_ bounds].size;
-  size.height += kLabelTopPadding;
 
   // Final inputField_ sizing/spacing depends on a TODO(estade) in Views code.
   if (![inputField_ isHidden]) {
     size.height = std::max(size.height, NSHeight([inputField_ frame]));
     size.width += NSWidth([inputField_ frame])  + kAroundTextPadding;
   }
 
   size.height += kTopPadding;
 
   return size;
 }
 
 - (void)performLayout {
   NSRect bounds = [[self view] bounds];
   NSSize preferredContainerSize = [self preferredSize];
   // width is externally determined.
   preferredContainerSize.width = NSWidth(bounds);
 
   NSRect spacerFrame = NSMakeRect(0, preferredContainerSize.height - 1,
                                   preferredContainerSize.width, 1);
 
   NSRect labelFrame = [label_ bounds];
   labelFrame.origin.x = NSMinX(bounds);
-  labelFrame.origin.y =
-      NSMaxY(bounds) - kLabelTopPadding - NSHeight(labelFrame) - kTopPadding;
+  labelFrame.origin.y = NSMaxY(bounds) - NSHeight(labelFrame) - kTopPadding;
 
   // Position input field - top is aligned to top of label field.
   if (![inputField_ isHidden]) {
     NSRect inputfieldFrame = [inputField_ frame];
     inputfieldFrame.origin.x = NSMaxX(bounds) - NSWidth(inputfieldFrame);
-    inputfieldFrame.origin.y =
-        NSMaxY(labelFrame) + kLabelTopPadding - NSHeight(inputfieldFrame);
+    inputfieldFrame.origin.y = NSMaxY(labelFrame) + NSHeight(inputfieldFrame);
     [inputField_ setFrameOrigin:inputfieldFrame.origin];
 
     // Due to fixed width, fields are guaranteed to not overlap.
     DCHECK_LE(NSMaxX(labelFrame), NSMinX(inputfieldFrame));
   }
 
   [spacer_ setFrame:spacerFrame];
   [label_ setFrame:labelFrame];
   [[self view] setFrameSize:preferredContainerSize];
 }
 
 @end