Move UserInfo to user_manager.

Move UserInfo to user_manager.

Since user_manager::UserInfo is used on non-chromeos ash implementation, defined part of user_manager component that is included on cross-platform ash.

BUG=387614
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=283087

[Nikita (slow), 2014-07-09 11:37:33]

oshima@ - please review whole CL

OWNERS:
skuhne@: chrome/browser/ui/ash/multi_user/*
msw@: chrome/browser/ui/views/*
blundell@: components/*

So plan is to use user_manager in ash and athena.

[Nikita (slow), 2014-07-09 11:49:51]

+skuhne@ for chrome/browser/ui/ash/multi_user/*

[blundell, 2014-07-09 12:23:49]

https://codereview.chromium.org/379803004/diff/1/components/user_manager.gypi
File components/user_manager.gypi (right):

https://codereview.chromium.org/379803004/diff/1/components/user_manager.gypi...
components/user_manager.gypi:14: 'user_manager/user_manager_export.h',
this should be under shared sources, no?

https://codereview.chromium.org/379803004/diff/1/components/user_manager.gypi...
components/user_manager.gypi:30: ['use_ash==1', {
is this condition necessary when the gypi is only include when use_ash==1?

[Nikita (slow), 2014-07-09 12:40:16]

https://codereview.chromium.org/379803004/diff/1/components/user_manager.gypi
File components/user_manager.gypi (right):

https://codereview.chromium.org/379803004/diff/1/components/user_manager.gypi...
components/user_manager.gypi:14: 'user_manager/user_manager_export.h',
On 2014/07/09 12:23:49, blundell wrote:
> this should be under shared sources, no?

Done.

https://codereview.chromium.org/379803004/diff/1/components/user_manager.gypi...
components/user_manager.gypi:30: ['use_ash==1', {
On 2014/07/09 12:23:49, blundell wrote:
> is this condition necessary when the gypi is only include when use_ash==1?

I was not sure where to leave this use_ash condition, removing from this file.

[blundell, 2014-07-09 12:41:55]

//components lgtm

[Mr4D (OOO till 08-26), 2014-07-09 14:07:59]

chrome/browser/ui/ash/multi_user lgtm

[msw, 2014-07-09 16:59:16]

chrome/browser/ui/views/* lgtm

[oshima, 2014-07-09 18:07:27]

lgtm

[Nikita (slow), 2014-07-09 19:10:04]

The CQ bit was checked by nkostylev@chromium.org

[commit-bot: I haz the power, 2014-07-09 19:11:58]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nkostylev@chromium.org/379803004/10001

[Nikita (slow), 2014-07-09 20:10:11]

The CQ bit was checked by nkostylev@chromium.org

[commit-bot: I haz the power, 2014-07-09 20:13:00]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nkostylev@chromium.org/379803004/30001

[commit-bot: I haz the power, 2014-07-09 23:27:21]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_compile_...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-07-10 00:04:42]

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

[commit-bot: I haz the power, 2014-07-10 00:04:43]

Try jobs failed on following builders:
  win_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_compile_...)

[Nikita (slow), 2014-07-14 18:41:21]

The CQ bit was checked by nkostylev@chromium.org

[commit-bot: I haz the power, 2014-07-14 18:42:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nkostylev@chromium.org/379803004/300001

[commit-bot: I haz the power, 2014-07-14 23:48:34]

Change committed as 283087