Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(211)

Side by Side Diff: ash/public/interfaces/login_user_info.mojom

Issue 2937553002: Create Mojo Struct for user information used in login/lock screen. (Closed)
Patch Set: comments and rebase Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 module ash.mojom;
6
7 import "ash/public/interfaces/user_info.mojom";
8 import "mojo/common/string16.mojom";
9
10 // Supported authentication types. Keep in sync with enum in
11 // screenlock_bridge.h
12 enum AuthType {
13 OFFLINE_PASSWORD,
14 ONLINE_SIGN_IN,
15 NUMERIC_PIN,
16 USER_CLICK,
17 EXPAND_THEN_USER_CLICK,
18 FORCE_OFFLINE_PASSWORD,
19 };
20
21 // Information about the custom icon in the user pod.
22 struct UserPodCustomIconOptions {
23 string id;
24 mojo.common.mojom.String16 tooltip;
25 bool autoshow_tooltip;
26 mojo.common.mojom.String16 aria_label;
27 bool hardlock_on_click;
28 bool is_trial_run;
29 };
30
31 // Info about a user in login/lock screen.
32 struct LoginUserInfo {
33 // User's basic information including account id, email, avatar etc.
34 UserInfo basic_user_info;
35
36 // True if this user is a desktop user.
jdufault 2017/06/14 02:20:42 What is a desktop user? My best guess is that we'r
xiaoyinh(OOO Sep 11-29) 2017/06/15 18:39:37 Removed, thanks!
37 bool is_desktop_user;
38
39 // Supported authentication type for this user.
jdufault 2017/06/14 02:20:42 What about // What method the user can use to s
xiaoyinh(OOO Sep 11-29) 2017/06/15 18:39:37 Changed. Thanks!
40 AuthType auth_type;
41
42 // True if this user has already signed in.
43 bool is_signed_in;
44
45 // True if this user is the device owner.
46 bool is_owner;
jdufault 2017/06/14 02:20:42 is_owner => is_device_owner
xiaoyinh(OOO Sep 11-29) 2017/06/15 18:39:37 Done.
47
48 // True if fingerprint unlock is allowed for this user.
49 bool allow_fingerprint_unlock;
50
51 // True if multi-profiles sign in is allowed for this user.
52 bool is_multiprofile_allowed;
53
54 // Enforced policy for multi-profiles sign in.
55 string multiprofile_policy;
jdufault 2017/06/14 02:20:42 Do we know what values this can be? Can it be an e
xiaoyinh(OOO Sep 11-29) 2017/06/15 18:39:37 The values seem to be harded coded here: https://c
jdufault 2017/06/15 19:00:02 Yes, an enum is much better. Can you refactor that
56
57 // True if this user can be removed.
58 bool can_remove;
59 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698