MD Settings: CrOS: Users: Fix names

MD Settings: CrOS: Users: Fix names

This CL:
* Uses the correct display name for users.
* Uses proper string substitution for l10n of owner user names.

BUG=706333
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2797163005
Cr-Commit-Position: refs/heads/master@{#462548}
Committed: https://chromium.googlesource.com/chromium/src/+/10c922d0770597a995a8609f40dd072c3521a96e

[stevenjb, 2017-04-06 02:22:44]

Description was changed from

==========
MD Settings: CrOS: Users: Fix names

This CL:
* Uses the correct display name for users.
* Uses proper string substitution for l10n of owner user names.

BUG=706333
==========

to

==========
MD Settings: CrOS: Users: Fix names

This CL:
* Uses the correct display name for users.
* Uses proper string substitution for l10n of owner user names.

BUG=706333
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[stevenjb, 2017-04-06 02:26:46]

stevenjb@chromium.org changed reviewers:
+ michaelpg@chromium.org

[michaelpg, 2017-04-06 14:50:29]

lgtm

https://codereview.chromium.org/2797163005/diff/20001/chrome/browser/chromeos...
File chrome/browser/chromeos/extensions/users_private/users_private_api.cc
(right):

https://codereview.chromium.org/2797163005/diff/20001/chrome/browser/chromeos...
chrome/browser/chromeos/extensions/users_private/users_private_api.cc:106: //
User is not associated with a gaia account.
what cases don't have gaia accounts or display names?

[stevenjb, 2017-04-06 16:27:58]

https://codereview.chromium.org/2797163005/diff/20001/chrome/browser/chromeos...
File chrome/browser/chromeos/extensions/users_private/users_private_api.cc
(right):

https://codereview.chromium.org/2797163005/diff/20001/chrome/browser/chromeos...
chrome/browser/chromeos/extensions/users_private/users_private_api.cc:106: //
User is not associated with a gaia account.
On 2017/04/06 14:50:29, michaelpg wrote:
> what cases don't have gaia accounts or display names?

I'm not really sure, just making sure we are consistent with previous behavior.
In theory it is possible to log into chromiumos with a non google account,
although I don't know whether that has been implemented in practice.

[stevenjb, 2017-04-06 16:28:02]

The CQ bit was checked by stevenjb@chromium.org

[commit-bot: I haz the power, 2017-04-06 16:33:31]

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-04-06 18:16:42]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491496082297450,
"parent_rev": "2ad62ba5846137528f1e75332c4f0f54dab0593e", "commit_rev":
"10c922d0770597a995a8609f40dd072c3521a96e"}

[commit-bot: I haz the power, 2017-04-06 18:17:59]

Description was changed from

==========
MD Settings: CrOS: Users: Fix names

This CL:
* Uses the correct display name for users.
* Uses proper string substitution for l10n of owner user names.

BUG=706333
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: CrOS: Users: Fix names

This CL:
* Uses the correct display name for users.
* Uses proper string substitution for l10n of owner user names.

BUG=706333
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2797163005
Cr-Commit-Position: refs/heads/master@{#462548}
Committed:
https://chromium.googlesource.com/chromium/src/+/10c922d0770597a995a8609f40dd...
==========

[commit-bot: I haz the power, 2017-04-06 18:18:01]

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