Desktop user manager: Adjustments for supervised users.

Desktop user manager: Adjustments for supervised users.

The changes are:
- Show a supervised user icon. Note that this has a different look than on CrOS, to fit in with the "lock" icon on desktop. (The final icon resource is still missing, but since this must land in time for M37 I'm already sending the code for review now, and will switch out the icon later.)
- Use the supervised user deletion warning text from CrOS also for supervised users on desktop.

Mocks at https://folio.googleplex.com/chrome-ux/mocks/064-chrome-away-from-chrome/supervised/mirror

BUG=372812, 310640
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274215

[Marc Treib, 2014-05-27 12:34:39]

Hi Nikita and Monica, PTAL!
[Nikita owns all the files in this CL, but adding Monica as the owner of the
desktop user manager as well!]

[Nikita (slow), 2014-05-27 13:29:58]

I'll let Denis review this since he's working on supervised users on Chrome OS.

[Denis Kuznetsov (DE-MUC), 2014-05-28 12:51:23]

lgtm

[noms (inactive), 2014-05-28 13:44:49]

lgtm

[Marc Treib, 2014-05-28 13:52:16]

Hi Nikita, can you review the change in user_manager.css, since neither Denis
nor Monica own it? Thanks!

[Nikita (slow), 2014-05-28 14:54:05]

https://chromiumcodereview.appspot.com/307483002/diff/40001/chrome/browser/re...
File chrome/browser/resources/user_manager/user_manager.css (right):

https://chromiumcodereview.appspot.com/307483002/diff/40001/chrome/browser/re...
chrome/browser/resources/user_manager/user_manager.css:82: left: 10px;
Please make this RTL friendly.

[Nikita (slow), 2014-05-28 14:55:14]

On 2014/05/28 14:54:05, Nikita Kostylev wrote:
>
https://chromiumcodereview.appspot.com/307483002/diff/40001/chrome/browser/re...
> File chrome/browser/resources/user_manager/user_manager.css (right):
> 
>
https://chromiumcodereview.appspot.com/307483002/diff/40001/chrome/browser/re...
> chrome/browser/resources/user_manager/user_manager.css:82: left: 10px;
> Please make this RTL friendly.

See chrome/browser/resources/login/user_pod_row.css

[Marc Treib, 2014-05-28 15:10:47]

https://chromiumcodereview.appspot.com/307483002/diff/40001/chrome/browser/re...
File chrome/browser/resources/user_manager/user_manager.css (right):

https://chromiumcodereview.appspot.com/307483002/diff/40001/chrome/browser/re...
chrome/browser/resources/user_manager/user_manager.css:82: left: 10px;
On 2014/05/28 14:54:06, Nikita Kostylev wrote:
> Please make this RTL friendly.

The supervised icon is supposed to take the place of the "locked" icon which is
defined just above, and which is also not adjusted for RTL. So I think it should
stay like this, and be changed only if the lock icon also becomes RTL-adjusted
at some point.

[Nikita (slow), 2014-05-28 15:13:49]

On 2014/05/28 15:10:47, treib wrote:
>
https://chromiumcodereview.appspot.com/307483002/diff/40001/chrome/browser/re...
> File chrome/browser/resources/user_manager/user_manager.css (right):
> 
>
https://chromiumcodereview.appspot.com/307483002/diff/40001/chrome/browser/re...
> chrome/browser/resources/user_manager/user_manager.css:82: left: 10px;
> On 2014/05/28 14:54:06, Nikita Kostylev wrote:
> > Please make this RTL friendly.
> 
> The supervised icon is supposed to take the place of the "locked" icon which
is
> defined just above, and which is also not adjusted for RTL. So I think it
should
> stay like this, and be changed only if the lock icon also becomes RTL-adjusted
> at some point.

Monica, do you want to take this task? :)

[noms (inactive), 2014-05-28 15:14:50]

Sure, I'll whip up a CL later on today.

On 2014/05/28 15:13:49, Nikita Kostylev wrote:
> On 2014/05/28 15:10:47, treib wrote:
> >
>
https://chromiumcodereview.appspot.com/307483002/diff/40001/chrome/browser/re...
> > File chrome/browser/resources/user_manager/user_manager.css (right):
> > 
> >
>
https://chromiumcodereview.appspot.com/307483002/diff/40001/chrome/browser/re...
> > chrome/browser/resources/user_manager/user_manager.css:82: left: 10px;
> > On 2014/05/28 14:54:06, Nikita Kostylev wrote:
> > > Please make this RTL friendly.
> > 
> > The supervised icon is supposed to take the place of the "locked" icon which
> is
> > defined just above, and which is also not adjusted for RTL. So I think it
> should
> > stay like this, and be changed only if the lock icon also becomes
RTL-adjusted
> > at some point.
> 
> Monica, do you want to take this task? :)

[Nikita (slow), 2014-05-29 07:54:58]

lgtm

[noms (inactive), 2014-05-29 16:49:45]

CL created here: https://codereview.chromium.org/303193002/
On 2014/05/29 07:54:58, Nikita Kostylev wrote:
> lgtm

[Marc Treib, 2014-06-02 07:56:06]

The CQ bit was checked by treib@chromium.org

[commit-bot: I haz the power, 2014-06-02 07:56:43]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/treib@chromium.org/307483002/50001

[commit-bot: I haz the power, 2014-06-02 12:38:28]

Change committed as 274215