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

Unified 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 side-by-side diff with in-line comments
Download patch
Index: ash/public/interfaces/login_user_info.mojom
diff --git a/ash/public/interfaces/login_user_info.mojom b/ash/public/interfaces/login_user_info.mojom
new file mode 100644
index 0000000000000000000000000000000000000000..18d22847e6f7508e596fdef06e9e7ee745a1b2da
--- /dev/null
+++ b/ash/public/interfaces/login_user_info.mojom
@@ -0,0 +1,59 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+module ash.mojom;
+
+import "ash/public/interfaces/user_info.mojom";
+import "mojo/common/string16.mojom";
+
+// Supported authentication types. Keep in sync with enum in
+// screenlock_bridge.h
+enum AuthType {
+ OFFLINE_PASSWORD,
+ ONLINE_SIGN_IN,
+ NUMERIC_PIN,
+ USER_CLICK,
+ EXPAND_THEN_USER_CLICK,
+ FORCE_OFFLINE_PASSWORD,
+};
+
+// Information about the custom icon in the user pod.
+struct UserPodCustomIconOptions {
+ string id;
+ mojo.common.mojom.String16 tooltip;
+ bool autoshow_tooltip;
+ mojo.common.mojom.String16 aria_label;
+ bool hardlock_on_click;
+ bool is_trial_run;
+};
+
+// Info about a user in login/lock screen.
+struct LoginUserInfo {
+ // User's basic information including account id, email, avatar etc.
+ UserInfo basic_user_info;
+
+ // 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!
+ bool is_desktop_user;
+
+ // 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!
+ AuthType auth_type;
+
+ // True if this user has already signed in.
+ bool is_signed_in;
+
+ // True if this user is the device owner.
+ 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.
+
+ // True if fingerprint unlock is allowed for this user.
+ bool allow_fingerprint_unlock;
+
+ // True if multi-profiles sign in is allowed for this user.
+ bool is_multiprofile_allowed;
+
+ // Enforced policy for multi-profiles sign in.
+ 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
+
+ // True if this user can be removed.
+ bool can_remove;
+};

Powered by Google App Engine
This is Rietveld 408576698