Remove old-avatar supervised user labels on non-Mac.

Remove old-avatar supervised user labels on non-Mac.

These are unused; the code to create them is never run because we no longer show
avatars on non-incognito windows except in CrOS with multiple sign-in, and in
that case AFAICT it's not possible to multiple-sign-in a supervised user.

This doesn't remove various resources (strings, images) since they're still used
by the Mac code.  I'm not able to tell whether than Mac code is also dead.

BUG=none
TEST=none

Committed: https://crrev.com/41bbc33878bb9cb1284c743c4db64203e9cd132d
Cr-Commit-Position: refs/heads/master@{#370826}

[Peter Kasting, 2016-01-20 23:49:46]

pkasting@chromium.org changed reviewers:
+ akuegel@chromium.org

[Peter Kasting, 2016-01-20 23:49:46]

Adrian, you did a lot of the very early work on the avatar label code, do you
know if there's a better owner now to review this?

[Adrian Kuegel, 2016-01-21 08:19:12]

akuegel@chromium.org changed reviewers:
+ treib@chromium.org

[Adrian Kuegel, 2016-01-21 08:19:13]

Marc Treib is probably the right owner.

[Adrian Kuegel, 2016-01-21 08:20:54]

Btw, thanks for the cleanup.

@Marc: I would guess the mac code is dead as well?

[Marc Treib, 2016-01-21 18:29:41]

lgtm, thanks!

Yup, the corresponding Mac code will be dead as well. I can make a CL to remove
it.

[Peter Kasting, 2016-01-21 22:03:13]

On 2016/01/21 18:29:41, Marc Treib wrote:
> lgtm, thanks!
> 
> Yup, the corresponding Mac code will be dead as well. I can make a CL to
remove
> it.

Cool.  Feel free to use me as a reviewer for removing strings, images, etc. --
everything but the actual ObjC changes.

[Peter Kasting, 2016-01-21 22:03:20]

The CQ bit was checked by pkasting@chromium.org

[commit-bot: I haz the power, 2016-01-21 22:05:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1614523002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1614523002/1

[commit-bot: I haz the power, 2016-01-21 23:28:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-21 23:30:28]

Description was changed from

==========
Remove old-avatar supervised user labels on non-Mac.

These are unused; the code to create them is never run because we no longer show
avatars on non-incognito windows except in CrOS with multiple sign-in, and in
that case AFAICT it's not possible to multiple-sign-in a supervised user.

This doesn't remove various resources (strings, images) since they're still used
by the Mac code.  I'm not able to tell whether than Mac code is also dead.

BUG=none
TEST=none
==========

to

==========
Remove old-avatar supervised user labels on non-Mac.

These are unused; the code to create them is never run because we no longer show
avatars on non-incognito windows except in CrOS with multiple sign-in, and in
that case AFAICT it's not possible to multiple-sign-in a supervised user.

This doesn't remove various resources (strings, images) since they're still used
by the Mac code.  I'm not able to tell whether than Mac code is also dead.

BUG=none
TEST=none

Committed: https://crrev.com/41bbc33878bb9cb1284c743c4db64203e9cd132d
Cr-Commit-Position: refs/heads/master@{#370826}
==========

[commit-bot: I haz the power, 2016-01-21 23:30:29]

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