Reproducible on 54.0.2840.6/8743.4.0 dev-channel Blaze

Pam, do you know who owns this?

What I'm currently seeing is a little different.  When the "Add supervised user" button is visible, TAB order goes
  ... > Add person > [...] > Add supervised user > Sys Tray > ...
When focus is on [...], the blue focus highlight is drawn around the button, but is partially hidden by the "Add supervised user" button (see 1st attached image).  When focus moves to "Add supervised user", that focus highlight disappears, and there is no visible focus anywhere.

I have a draft fix where "Add supervised user" focus is indicated with a 1px blue border (see 2nd attached image).  It's a little hard to see, but somewhat more obvious when it appears and disappears as focus is changing.  I'll need to know if this is an acceptable focus style, or what should be done instead.

+elizabethchiu@, who's been giving UX advice on other login page focus questions (Issue 412616).

NOTE: I think the original report is implying that focus should skip [...] when "Add supervised user" is visible.  I'm not sure this is correct, as focusing [...] + ENTER is currently the only way to hide "Add supervised user" with the keyboard.  When  Issue 525651  gets fixed, skipping [...] in this case would make more sense.

Deleted: focus_more.png 3.5 KB

	focus_more.png 3.5 KB View Download

Deleted: focus_addsu.png 3.4 KB

	focus_addsu.png 3.4 KB View Download

The attached is what the harmony focused mode looks like and if we can make the change to the other UI on the shelf, it would be good. Also please center the button with "add person" and the shelf. It looks off to me right now. 

https://docs.google.com/a/google.com/presentation/d/17CDjsa8u0rhgjV0zTCJMwY1z5BbaaXKiJqBL_N1s-9A/edit?usp=sharing

Deleted: Screen Shot 2016-12-09 at 4.18.12 PM.png 73.9 KB

	Screen Shot 2016-12-09 at 4.18.12 PM.png 73.9 KB View Download

There's a CL in the works:
https://codereview.chromium.org/2610193002
It adds a focus rect and vertically aligns the box, as described in Comment 5 (see attached).

There's also a CL under review for  Issue 525651  to activate the ESC key on the menu:
https://codereview.chromium.org/2609393003
(It also has TAB order skip the [...] button while [Add supervised user] is visible).

We're blocking Issue 615892, as that includes matching styles for other login shelf button focus rects.

Deleted: focus_addsu2.png 9.6 KB

	focus_addsu2.png 9.6 KB View Download

Regarding Comment 5... the [Add supervised user] menu/button appears to have been deliberately offset up and to the right by 5px.  Does anyone know why this was done?  The current draft of my CL is removing this offset, placing [Add supervised user] directly on top of the [...] button.  This can be seen in the attached screenshot, where it sits directly adjacent to the focus rect around [Add person].  When [Add supervised user] is visible, it may be closed by the ESC key, and [...] is removed from the TAB order (see Comment 6), so I see no problem with [...] being completely hidden.

elizabethchiu@ - Does this all look reasonable?

Deleted: focus_addsu3.png 10.1 KB

	focus_addsu3.png 10.1 KB View Download

That's reasonable to me. Is it possible to add space between the two for 8dp?

It's certainly possible... just want to verify that it's desirable.  The focus rects for all the other login shelf buttons (e.g., [Shut down][Browse as Guest][Add person][...]) are all adjacent (have no space between them).  I certainly don't know the specifics of MD design aesthetics, but here's what I was thinking:

When [Add supervised user] was offset up/right by 5px, it felt more like a menu.  By moving it down to cover [...], it feels more like a button, so my inclination would be to place it more like a button (i.e. exactly overlay [...]).

Also, adding that 8px space means restoring some special-case RTL code that I had removed as unnecessary.

All that said, if adding the 8px is the right design decision, it's no real problem :-)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c5a91c9388a44ff129f6234a84a399b1abd97cba

commit c5a91c9388a44ff129f6234a84a399b1abd97cba
Author: glevin <glevin@chromium.org>
Date: Mon Feb 06 20:35:29 2017

Focus style for Add supervised user btn

BUG= 642330 
TEST=On login page, click More options [...] button, and TAB through
elements.  Blue focus rect should be seen on Add supervised user button.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2610193002
Cr-Commit-Position: refs/heads/master@{#448377}

[modify] https://crrev.com/c5a91c9388a44ff129f6234a84a399b1abd97cba/chrome/browser/resources/chromeos/login/header_bar.css

I went ahead and committed this w/o the 8px space mentioned in Comment 8.  It can be added back separately if the desire for it outweighs the thoughts in Comment 9.

Sounds good to me.

Verified on ChromeOS 9287.0.0, 58.0.3011.0