cros: Pull user info from session_controller_client.mojom into separate mojom.

cros: Pull user info from session_controller_client.mojom into separate mojom.

User info needed by login/lock screen will be provided from mutliple
sources, not just session_controller (ie, login uses user_manager, lock
uses session_controller).

BUG=719015
Review-Url: https://codereview.chromium.org/2911493002
Cr-Commit-Position: refs/heads/master@{#475193}
Committed: https://chromium.googlesource.com/chromium/src/+/0933f4586224dd6de793b6118b763910e0a19d92

[jdufault, 2017-05-25 21:13:28]

The CQ bit was checked by jdufault@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-25 21:14:12]

Dry run: CQ is trying da patch.

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

[jdufault, 2017-05-25 22:56:14]

Description was changed from

==========
cros: Pull user info from session_controller_client.mojom into separate mojom.

User info needed by login/lock screen will be provided from mutliple
sources, not just session_controller (ie, login uses user_manager, lock
uses session_controller).

BUG=719015
==========

to

==========
cros: Pull user info from session_controller_client.mojom into separate mojom.

User info needed by login/lock screen will be provided from mutliple
sources, not just session_controller (ie, login uses user_manager, lock
uses session_controller).

BUG=719015
==========

[jdufault, 2017-05-25 22:56:14]

jdufault@chromium.org changed reviewers:
+ xiyuan@chromium.org

[jdufault, 2017-05-25 22:56:34]

xiyuan@ PTAL

[xiyuan, 2017-05-25 23:08:46]

lgtm

https://codereview.chromium.org/2911493002/diff/1/ash/public/interfaces/typem...
File ash/public/interfaces/typemaps.gni (right):

https://codereview.chromium.org/2911493002/diff/1/ash/public/interfaces/typem...
ash/public/interfaces/typemaps.gni:6:
"//ash/public/interfaces/user_info.typemap",
keep the list alpha-sorted

[commit-bot: I haz the power, 2017-05-25 23:28:25]

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

[commit-bot: I haz the power, 2017-05-25 23:28:26]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[jdufault, 2017-05-26 17:31:46]

The CQ bit was checked by jdufault@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-26 17:32:35]

Dry run: CQ is trying da patch.

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

[jdufault, 2017-05-26 17:33:01]

https://codereview.chromium.org/2911493002/diff/1/ash/public/interfaces/typem...
File ash/public/interfaces/typemaps.gni (right):

https://codereview.chromium.org/2911493002/diff/1/ash/public/interfaces/typem...
ash/public/interfaces/typemaps.gni:6:
"//ash/public/interfaces/user_info.typemap",
On 2017/05/25 23:08:46, xiyuan wrote:
> keep the list alpha-sorted

Oops, done. I was thinking git cl format would do it (it auto-orders gn files).

[jdufault, 2017-05-26 17:35:46]

jdufault@chromium.org changed reviewers:
+ jamescook@chromium.org, tsepez@chromium.org

[jdufault, 2017-05-26 17:37:01]

tsepez@: PTAL at mojom changes
jamescook@: PTAL (need owner approval)

[Tom Sepez, 2017-05-26 17:57:59]

RS LGTM on moving existing stuff around.

[commit-bot: I haz the power, 2017-05-26 19:11:34]

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

[commit-bot: I haz the power, 2017-05-26 19:11:35]

Dry run: This issue passed the CQ dry run.

[James Cook, 2017-05-26 23:35:47]

LGTM

[jdufault, 2017-05-26 23:39:27]

The CQ bit was checked by jdufault@chromium.org

[jdufault, 2017-05-26 23:39:32]

The patchset sent to the CQ was uploaded after l-g-t-m from xiyuan@chromium.org
Link to the patchset: https://codereview.chromium.org/2911493002/#ps20001
(title: "Fix ordering in typemaps")

[commit-bot: I haz the power, 2017-05-26 23:39:45]

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, 2017-05-26 23:44:28]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495841967513000,
"parent_rev": "cd559ca9b161b9bf1cf77b7f387faf4f6616c945", "commit_rev":
"0933f4586224dd6de793b6118b763910e0a19d92"}

[commit-bot: I haz the power, 2017-05-26 23:44:37]

Description was changed from

==========
cros: Pull user info from session_controller_client.mojom into separate mojom.

User info needed by login/lock screen will be provided from mutliple
sources, not just session_controller (ie, login uses user_manager, lock
uses session_controller).

BUG=719015
==========

to

==========
cros: Pull user info from session_controller_client.mojom into separate mojom.

User info needed by login/lock screen will be provided from mutliple
sources, not just session_controller (ie, login uses user_manager, lock
uses session_controller).

BUG=719015

Review-Url: https://codereview.chromium.org/2911493002
Cr-Commit-Position: refs/heads/master@{#475193}
Committed:
https://chromium.googlesource.com/chromium/src/+/0933f4586224dd6de793b6118b76...
==========

[commit-bot: I haz the power, 2017-05-26 23:44:38]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/0933f4586224dd6de793b6118b76...