Include a active state on the avatar button when menu is open. Specs are identical to that of the pressed state. PREVIEW https://folio.googleplex.com/chrome-ux-specs-and-sources/Chrome%20Desktop%20Sign%20In/user_menu-button/P%20-%20macOSX#%2F04-signed-in-pressed.png%3Fz=width SPEC https://folio.googleplex.com/chrome-ux-specs-and-sources/Chrome%20Desktop%20Sign%20In/user_menu-button#%2FSPEC%20-%20user-menu-button.png%3Fz=width
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/150c941e115f73ac37c7ae2b840d2420f1c6f45e commit 150c941e115f73ac37c7ae2b840d2420f1c6f45e Author: spqchan <spqchan@chromium.org> Date: Fri Oct 28 16:58:03 2016 [Mac] Clean up AvatarButtonController Checks for Material Design had recently been removed. However, there's still a lot of leftover code. This CL cleans up AvatarButtonController so that it will be easier to add new things to it. BUG= 654876 Review-Url: https://codereview.chromium.org/2455323002 Cr-Commit-Position: refs/heads/master@{#428395} [modify] https://crrev.com/150c941e115f73ac37c7ae2b840d2420f1c6f45e/chrome/browser/ui/cocoa/profiles/avatar_button_controller.mm
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d7387b36c1f7bf70cedbf7a4661386a2c336bdc9 commit d7387b36c1f7bf70cedbf7a4661386a2c336bdc9 Author: spqchan <spqchan@chromium.org> Date: Thu Nov 03 17:35:33 2016 [Mac] Implement an active state for the avatar button BUG= 654876 Review-Url: https://codereview.chromium.org/2459073003 Cr-Commit-Position: refs/heads/master@{#429633} [modify] https://crrev.com/d7387b36c1f7bf70cedbf7a4661386a2c336bdc9/chrome/browser/ui/cocoa/profiles/avatar_base_controller.h [modify] https://crrev.com/d7387b36c1f7bf70cedbf7a4661386a2c336bdc9/chrome/browser/ui/cocoa/profiles/avatar_button.h [modify] https://crrev.com/d7387b36c1f7bf70cedbf7a4661386a2c336bdc9/chrome/browser/ui/cocoa/profiles/avatar_button.mm [modify] https://crrev.com/d7387b36c1f7bf70cedbf7a4661386a2c336bdc9/chrome/browser/ui/cocoa/profiles/avatar_button_controller.h [modify] https://crrev.com/d7387b36c1f7bf70cedbf7a4661386a2c336bdc9/chrome/browser/ui/cocoa/profiles/avatar_button_controller.mm
Comment 1 by spqc...@chromium.org
, Oct 27 2016