[Material][Mac] Update the 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"
 #include "chrome/browser/profiles/profile_attributes_storage.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/profiles/profiles_state.h"
 #include "chrome/browser/themes/theme_service.h"
 #include "chrome/browser/themes/theme_service_factory.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_window.h"
 #import "chrome/browser/ui/cocoa/browser_window_controller.h"
 #import "chrome/browser/ui/cocoa/profiles/avatar_button.h"
 #include "chrome/grit/generated_resources.h"
 #include "chrome/grit/theme_resources.h"
 #include "components/signin/core/common/profile_management_switches.h"
+#include "skia/ext/skia_utils_mac.h"
 #import "ui/base/cocoa/appkit_utils.h"
 #include "ui/base/l10n/l10n_util_mac.h"
 #include "ui/base/material_design/material_design_controller.h"
 #include "ui/base/nine_image_painter_factory.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/color_palette.h"
 #include "ui/gfx/image/image_skia_operations.h"
 #include "ui/gfx/image/image_skia_util_mac.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/gfx/vector_icons_public.h"
 
 namespace {
 
-// NSButtons have a default padding of 5px. This button should have a padding
-// of 8px.
-const CGFloat kButtonExtraPadding = 8 - 5;
-const CGFloat kButtonHeight = 28;
-
 const ui::NinePartImageIds kNormalBorderImageIds =
     IMAGE_GRID(IDR_AVATAR_NATIVE_BUTTON_NORMAL);
 const ui::NinePartImageIds kHoverBorderImageIds =
     IMAGE_GRID(IDR_AVATAR_NATIVE_BUTTON_HOVER);
 const ui::NinePartImageIds kPressedBorderImageIds =
     IMAGE_GRID(IDR_AVATAR_NATIVE_BUTTON_PRESSED);
 const ui::NinePartImageIds kThemedBorderImageIds =
     IMAGE_GRID(IDR_AVATAR_THEMED_MAC_BUTTON_NORMAL);
 
 NSImage* GetImageFromResourceID(int resourceId) {
   return ui::ResourceBundle::GetSharedInstance().GetNativeImageNamed(
       resourceId).ToNSImage();
 }
 
+const SkColor kMaterialButtonHoverColor = SkColorSetARGB(20, 0, 0, 0);
+const SkColor kMaterialButtonPressedColor = SkColorSetARGB(31, 0, 0, 0);
+const SkColor kMaterialAvatarIconColor = SkColorSetRGB(0x5a, 0x5a, 0x5a);
+
+CGFloat ButtonHeight() {
+  const CGFloat kButtonHeight = 28;
+  const CGFloat kMaterialButtonHeight = 24;
+  return ui::MaterialDesignController::IsModeMaterial() ? kMaterialButtonHeight
+                                                        : kButtonHeight;
+}
+
+// NSButtons have a default padding of 5px. Non-MD buttons should have a
+// padding of 8px. Meanwhile, MD buttons should have a padding of 6px.
+CGFloat ButtonExtraPadding() {
+  const CGFloat kDefaultPadding = 5;
+  const CGFloat kButtonExtraPadding = 8 - kDefaultPadding;
+  const CGFloat kMaterialButtonExtraPadding = 6 - kDefaultPadding;
+
+  return ui::MaterialDesignController::IsModeMaterial()
+             ? kMaterialButtonExtraPadding
+             : kButtonExtraPadding;
+}
+
+// Extra padding for the MD signed out avatar button.
+const CGFloat kMaterialSignedOutWidthPadding = 2;
+
 }  // 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;
 
 @end
 
 @implementation CustomThemeButtonCell
 - (id)initWithThemedWindow:(BOOL)isThemedWindow {
   if ((self = [super init])) {
     isThemedWindow_ = isThemedWindow;
     hasError_ = false;
   }
   return self;
 }
 
 - (NSSize)cellSize {
   NSSize buttonSize = [super cellSize];
 
   // An image and no error means we are drawing the generic button, which
   // is square. Otherwise, we are displaying the profile's name and an
   // optional authentication error icon.
   if ([self image] && !hasError_) {
-    buttonSize.width = kButtonHeight;
+    buttonSize.width = ButtonHeight();
+    if (ui::MaterialDesignController::IsModeMaterial())
+      buttonSize.width += kMaterialSignedOutWidthPadding;
   } else {
-    buttonSize.width += 2 * kButtonExtraPadding;
+    buttonSize.width += 2 * ButtonExtraPadding();
   }
-  buttonSize.height = kButtonHeight;
+  buttonSize.height = ButtonHeight();
   return buttonSize;
 }
 
 - (void)drawInteriorWithFrame:(NSRect)frame inView:(NSView*)controlView {
-  NSRect frameAfterPadding = NSInsetRect(frame, kButtonExtraPadding, 0);
+  NSRect frameAfterPadding = NSInsetRect(frame, ButtonExtraPadding(), 0);
   [super drawInteriorWithFrame:frameAfterPadding inView:controlView];
 }
 
 - (void)drawImage:(NSImage*)image
         withFrame:(NSRect)frame
            inView:(NSView*)controlView {
   // The image used in the generic button case as well as the material-designed
   // error icon both need to be shifted down slightly to be centered correctly.
   // TODO(noms): When the assets are fixed, remove this latter offset.
   if (!hasError_ || switches::IsMaterialDesignUserMenu())
     frame = NSOffsetRect(frame, 0, 1);
   [super drawImage:image withFrame:frame inView:controlView];
 }
 
 - (void)drawBezelWithFrame:(NSRect)frame
                     inView:(NSView*)controlView {
   HoverState hoverState =
       [base::mac::ObjCCastStrict<AvatarButton>(controlView) hoverState];
-  ui::NinePartImageIds imageIds = kNormalBorderImageIds;
-  if (isThemedWindow_)
-    imageIds = kThemedBorderImageIds;
 
-  if (hoverState == kHoverStateMouseDown)
-    imageIds = kPressedBorderImageIds;
-  else if (hoverState == kHoverStateMouseOver)
-    imageIds = kHoverBorderImageIds;
-  ui::DrawNinePartImage(frame, imageIds, NSCompositeSourceOver, 1.0, true);
+  if (ui::MaterialDesignController::IsModeMaterial()) {
+    NSColor* backgroundColor = nil;
+    if (hoverState == kHoverStateMouseDown) {
+      backgroundColor = skia::SkColorToSRGBNSColor(kMaterialButtonPressedColor);
+    } else if (hoverState == kHoverStateMouseOver) {
+      backgroundColor = skia::SkColorToSRGBNSColor(kMaterialButtonHoverColor);
+    }
+
+    if (backgroundColor) {
+      [backgroundColor set];
+      NSBezierPath* path = [NSBezierPath bezierPathWithRoundedRect:frame
+                                                           xRadius:2.0f
+                                                           yRadius:2.0f];
+      [path fill];
+    }
+  } else {
+    ui::NinePartImageIds imageIds = kNormalBorderImageIds;
+    if (isThemedWindow_)
+      imageIds = kThemedBorderImageIds;
+
+    if (hoverState == kHoverStateMouseDown)
+      imageIds = kPressedBorderImageIds;
+    else if (hoverState == kHoverStateMouseOver)
+      imageIds = kHoverBorderImageIds;
+    ui::DrawNinePartImage(frame, imageIds, NSCompositeSourceOver, 1.0, true);
+  }
 }
 
 - (void)drawFocusRingMaskWithFrame:(NSRect)frame inView:(NSView*)view {
   // Match the bezel's shape.
   [[NSBezierPath bezierPathWithRoundedRect:NSInsetRect(frame, 2, 2)
                                    xRadius:2
                                    yRadius:2] fill];
 }
 
 - (void)setIsThemedWindow:(BOOL)isThemedWindow {
@@ skipping 24 matching lines @@
 
 - (id)initWithBrowser:(Browser*)browser {
   if ((self = [super initWithBrowser:browser])) {
     ThemeService* themeService =
         ThemeServiceFactory::GetForProfile(browser->profile());
     isThemedWindow_ = !themeService->UsingSystemTheme();
 
     AvatarButton* avatarButton =
         [[AvatarButton alloc] initWithFrame:NSZeroRect];
     button_.reset(avatarButton);
-    base::scoped_nsobject<CustomThemeButtonCell> cell(
+
+    base::scoped_nsobject<NSButtonCell> cell(
         [[CustomThemeButtonCell alloc] initWithThemedWindow:isThemedWindow_]);
+
     [avatarButton setCell:cell.get()];
 
     [avatarButton setWantsLayer:YES];
     [self setView:avatarButton];
 
     [avatarButton setBezelStyle:NSShadowlessSquareBezelStyle];
     [avatarButton setButtonType:NSMomentaryChangeButton];
     if (switches::IsMaterialDesignUserMenu())
       [[avatarButton cell] setHighlightsBy:NSNoCellMask];
     [avatarButton setBordered:YES];
@@ skipping 66 matching lines @@
       !storage.GetAllProfilesAttributes().front()->IsAuthenticated();
 
 
   NSString* buttonTitle = base::SysUTF16ToNSString(useGenericButton ?
       base::string16() :
       profiles::GetAvatarButtonTextForProfile(browser_->profile()));
   [[button_ cell] setHasError:hasError_ withTitle:buttonTitle];
 
   AvatarButton* button =
       base::mac::ObjCCastStrict<AvatarButton>(button_);
+
   if (useGenericButton) {
-    [button setDefaultImage:GetImageFromResourceID(
-        IDR_AVATAR_NATIVE_BUTTON_AVATAR)];
-    [button setHoverImage:GetImageFromResourceID(
-        IDR_AVATAR_NATIVE_BUTTON_AVATAR_HOVER)];
-    [button setPressedImage:GetImageFromResourceID(
-        IDR_AVATAR_NATIVE_BUTTON_AVATAR_PRESSED)];
-    // This is a workaround for an issue in the HoverImageButton where the
-    // button is initially sized incorrectly unless a default image is provided.
-    // See crbug.com/298501.
-    [button setImage:GetImageFromResourceID(IDR_AVATAR_NATIVE_BUTTON_AVATAR)];
+    if (ui::MaterialDesignController::IsModeMaterial()) {
+      NSImage* avatarIcon = NSImageFromImageSkia(
+          gfx::CreateVectorIcon(gfx::VectorIconId::USER_ACCOUNT_AVATAR, 18,
+                                kMaterialAvatarIconColor));
+      [button setDefaultImage:avatarIcon];
+      [button setHoverImage:nil];
+      [button setPressedImage:nil];
+    } else {
+      [button setDefaultImage:GetImageFromResourceID(
+                                  IDR_AVATAR_NATIVE_BUTTON_AVATAR)];
+      [button setHoverImage:GetImageFromResourceID(
+                                IDR_AVATAR_NATIVE_BUTTON_AVATAR_HOVER)];
+      [button setPressedImage:GetImageFromResourceID(
+                                  IDR_AVATAR_NATIVE_BUTTON_AVATAR_PRESSED)];
+      // This is a workaround for an issue in the HoverImageButton where the
+      // button is initially sized incorrectly unless a default image is
+      // provided.
+      // See crbug.com/298501.
+      [button setImage:GetImageFromResourceID(IDR_AVATAR_NATIVE_BUTTON_AVATAR)];
+    }
     [button setImagePosition:NSImageOnly];
   } else if (hasError_) {
+    BOOL isMaterial = ui::MaterialDesignController::IsModeMaterial();
     NSImage* errorIcon =
-        switches::IsMaterialDesignUserMenu()
+        isMaterial
             ? NSImageFromImageSkia(gfx::CreateVectorIcon(
                   gfx::VectorIconId::SYNC_PROBLEM, 16, gfx::kGoogleRed700))
             : GetImageFromResourceID(IDR_ICON_PROFILES_AVATAR_BUTTON_ERROR);
     [button setDefaultImage:errorIcon];
     [button setHoverImage:nil];
     [button setPressedImage:nil];
     [button setImage:errorIcon];
-    [button setImagePosition:switches::IsMaterialDesignUserMenu()
-                                 ? NSImageLeft
-                                 : NSImageRight];
+    [button setImagePosition:isMaterial ? NSImageLeft : NSImageRight];
   } else {
     [button setDefaultImage:nil];
     [button setHoverImage:nil];
     [button setPressedImage:nil];
     [button setImagePosition:NSNoImage];
   }
 
   base::scoped_nsobject<NSMutableParagraphStyle> paragraphStyle(
       [[NSMutableParagraphStyle alloc] init]);
   [paragraphStyle setAlignment:NSLeftTextAlignment];
@@ skipping 15 matching lines @@
         layoutSubviews];
   }
 }
 
 - (void)setErrorStatus:(BOOL)hasError {
   hasError_ = hasError;
   [self updateAvatarButtonAndLayoutParent:YES];
 }
 
 @end