[Material][Mac] Update the User Account Button

[Material][Mac] Update the User Account Button

BUG=590917
Committed: https://crrev.com/1f0a7bb629f3ad9464aba375a20a59e5d23ca23b
Cr-Commit-Position: refs/heads/master@{#416382}

[spqchan, 2016-08-26 23:42:45]

Description was changed from

==========
[Material][Mac] Update the User Account Button

BUG=590917
==========

to

==========
[Material][Mac] Update the User Account Button

BUG=590917
==========

[spqchan, 2016-08-26 23:42:45]

spqchan@chromium.org changed reviewers:
+ shrike@chromium.org

[spqchan, 2016-08-26 23:42:51]

PTAL

[shrike, 2016-08-31 18:35:53]

Hey spqchan@ - I'm having trouble applying the patch. Does it need to be
rebased?

[spqchan, 2016-08-31 22:55:03]

On 2016/08/31 18:35:53, shrike wrote:
> Hey spqchan@ - I'm having trouble applying the patch. Does it need to be
> rebased?

Rebased and fixed conflicts

[shrike, 2016-09-01 00:05:14]

Hi spqchan@,

For whatever reason I cannot log in to test how the button looks when you're
logged in. You should go ahead and proceed with landing the change and we'll
verify it all looks good in Canary (before cherry-picking back).

One small problem I did notice was the icon appears to be too small. On my
Retina machine the icon fits in a 28x28 box but looking at the spec in Photoshop
the size is 34x34.

[spqchan, 2016-09-01 19:49:26]

On 2016/09/01 00:05:14, shrike wrote:
> Hi spqchan@,
> 
> For whatever reason I cannot log in to test how the button looks when you're
> logged in. You should go ahead and proceed with landing the change and we'll
> verify it all looks good in Canary (before cherry-picking back).
> 
> One small problem I did notice was the icon appears to be too small. On my
> Retina machine the icon fits in a 28x28 box but looking at the spec in
Photoshop
> the size is 34x34.

You don't have to log in. You can just add a new profile and it will
automatically become "Person 1"
Also, according to the specs, the icon should be in a 24x28 box.

[spqchan, 2016-09-01 19:49:56]

spqchan@chromium.org changed reviewers:
+ avi@chromium.org

[spqchan, 2016-09-01 19:49:56]

+avi for OWNER review

[Avi (use Gerrit), 2016-09-01 19:59:01]

lgtm

[spqchan, 2016-09-01 20:43:30]

On 2016/09/01 19:59:01, Avi wrote:
> lgtm

thanks!

[spqchan, 2016-09-01 20:43:33]

The CQ bit was checked by spqchan@chromium.org

[commit-bot: I haz the power, 2016-09-01 20:44:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[spqchan, 2016-09-01 20:57:23]

The CQ bit was unchecked by spqchan@chromium.org

[spqchan, 2016-09-01 21:17:46]

The CQ bit was checked by spqchan@chromium.org

[spqchan, 2016-09-01 21:17:46]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org
Link to the patchset: https://codereview.chromium.org/2286993002/#ps100001
(title: "Increase icon size")

[commit-bot: I haz the power, 2016-09-01 21:18:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-01 22:51:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-01 22:51:36]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[spqchan, 2016-09-01 22:55:13]

The CQ bit was checked by spqchan@chromium.org

[commit-bot: I haz the power, 2016-09-01 22:55:46]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-02 00:12:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-02 00:12:52]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[spqchan, 2016-09-02 17:54:19]

The CQ bit was checked by spqchan@chromium.org

[spqchan, 2016-09-02 17:54:19]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org
Link to the patchset: https://codereview.chromium.org/2286993002/#ps120001
(title: "fixed test")

[commit-bot: I haz the power, 2016-09-02 17:54:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-02 20:35:23]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-02 20:35:24]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[spqchan, 2016-09-02 20:41:00]

The CQ bit was checked by spqchan@chromium.org

[commit-bot: I haz the power, 2016-09-02 20:41:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-02 22:08:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-02 22:08:21]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[spqchan, 2016-09-02 22:14:17]

The CQ bit was checked by spqchan@chromium.org

[commit-bot: I haz the power, 2016-09-02 22:14:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-02 23:19:26]

Description was changed from

==========
[Material][Mac] Update the User Account Button

BUG=590917
==========

to

==========
[Material][Mac] Update the User Account Button

BUG=590917
==========

[commit-bot: I haz the power, 2016-09-02 23:19:27]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-09-02 23:20:57]

Description was changed from

==========
[Material][Mac] Update the User Account Button

BUG=590917
==========

to

==========
[Material][Mac] Update the User Account Button

BUG=590917

Committed: https://crrev.com/1f0a7bb629f3ad9464aba375a20a59e5d23ca23b
Cr-Commit-Position: refs/heads/master@{#416382}
==========

[commit-bot: I haz the power, 2016-09-02 23:20:58]

Patchset 7 (id:??) landed as
https://crrev.com/1f0a7bb629f3ad9464aba375a20a59e5d23ca23b
Cr-Commit-Position: refs/heads/master@{#416382}