[Material][Mac] Adjust User Account Button

chrome/browser/ui/cocoa/profiles/avatar_button_controller.mm

 // Copyright 2014 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/profiles/avatar_button_controller.h"
 
 #include "base/mac/foundation_util.h"
 #include "base/strings/sys_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile_attributes_entry.h"
@@ skipping 56 matching lines @@
   const CGFloat kMaterialButtonExtraPadding = 6 - kDefaultPadding;
 
   return ui::MaterialDesignController::IsModeMaterial()
              ? kMaterialButtonExtraPadding
              : kButtonExtraPadding;
 }
 
 // Extra padding for the MD signed out avatar button.
 const CGFloat kMaterialSignedOutWidthPadding = 2;
 
+// Kern value for the MD avatar button title.
+const CGFloat kMaterialTitleKern = 0.25;
+
 }  // namespace
 
 // Button cell with a custom border given by a set of nine-patch image grids.
 @interface CustomThemeButtonCell : NSButtonCell {
  @private
    BOOL isThemedWindow_;
    BOOL hasError_;
 }
 - (void)setIsThemedWindow:(BOOL)isThemedWindow;
 - (void)setHasError:(BOOL)hasError withTitle:(NSString*)title;
@@ skipping 180 matching lines @@
   if (browser_->profile()->IsGuestSession() &&
       !ui::MaterialDesignController::IsModeMaterial()) {
     foregroundColor = [NSColor colorWithCalibratedWhite:1.0 alpha:0.9];
     [shadow setShadowColor:[NSColor colorWithCalibratedWhite:0.0 alpha:0.4]];
   } else if (!isThemedWindow_) {
     foregroundColor = [NSColor blackColor];
     [shadow setShadowColor:[NSColor colorWithCalibratedWhite:1.0 alpha:0.7]];
   } else {
     foregroundColor = [NSColor blackColor];
     [shadow setShadowColor:[NSColor colorWithCalibratedWhite:1.0 alpha:0.4]];
   }

[Avi (use Gerrit), 2016/09/20 00:35:43]

This shadow code from lines 262–280 are only used in the non-MD code path now.
Bring them into your new else block.

[spqchan, 2016/09/20 20:16:28]

Done.

 
   ProfileAttributesStorage& storage =
       g_browser_process->profile_manager()->GetProfileAttributesStorage();
   // If there is a single local profile, then use the generic avatar button
   // instead of the profile name. Never use the generic button if the active
   // profile is Guest.
   bool useGenericButton =
       !browser_->profile()->IsGuestSession() &&
       storage.GetNumberOfProfiles() == 1 &&
       !storage.GetAllProfilesAttributes().front()->IsAuthenticated();
@@ skipping 45 matching lines @@
     [button setDefaultImage:nil];
     [button setHoverImage:nil];
     [button setPressedImage:nil];
     [button setImagePosition:NSNoImage];
   }
 
   base::scoped_nsobject<NSMutableParagraphStyle> paragraphStyle(
       [[NSMutableParagraphStyle alloc] init]);
   [paragraphStyle setAlignment:NSLeftTextAlignment];
 
-  base::scoped_nsobject<NSAttributedString> attributedTitle(
-      [[NSAttributedString alloc]
-          initWithString:buttonTitle
-              attributes:@{ NSShadowAttributeName : shadow.get(),
-                            NSForegroundColorAttributeName : foregroundColor,
-                            NSParagraphStyleAttributeName : paragraphStyle }]);
-  [button_ setAttributedTitle:attributedTitle];
+  if (ui::MaterialDesignController::IsModeMaterial()) {
+    base::scoped_nsobject<NSAttributedString> attributedTitle(
+        [[NSAttributedString alloc]
+            initWithString:buttonTitle
+                attributes:@{
+                  NSForegroundColorAttributeName : foregroundColor,
+                  NSParagraphStyleAttributeName : paragraphStyle,
+                  NSKernAttributeName :
+                      [NSNumber numberWithFloat:kMaterialTitleKern]
+                }]);
+    [button_ setAttributedTitle:attributedTitle];
+  } else {
+    base::scoped_nsobject<NSAttributedString> attributedTitle(
+        [[NSAttributedString alloc]
+            initWithString:buttonTitle
+                attributes:@{
+                  NSShadowAttributeName : shadow.get(),
+                  NSForegroundColorAttributeName : foregroundColor,
+                  NSParagraphStyleAttributeName : paragraphStyle
+                }]);
+    [button_ setAttributedTitle:attributedTitle];
+  }
   [button_ sizeToFit];
 
   if (layoutParent) {
     // Because the width of the button might have changed, the parent browser
     // frame needs to recalculate the button bounds and redraw it.
     [[BrowserWindowController
         browserWindowControllerForWindow:browser_->window()->GetNativeWindow()]
         layoutSubviews];
   }
 }
 
 - (void)setErrorStatus:(BOOL)hasError {
   hasError_ = hasError;
   [self updateAvatarButtonAndLayoutParent:YES];
 }
 
 @end