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

Unified Diff: components/arc/common/auth.mojom

Issue 2771943003: arc: Skip GMS Sign-In in case Android is already signed-in. (Closed)
Patch Set: use sparse UMA Created 3 years, 9 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
« no previous file with comments | « chrome/browser/chromeos/arc/arc_optin_uma.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/arc/common/auth.mojom
diff --git a/components/arc/common/auth.mojom b/components/arc/common/auth.mojom
index c0617259ee98425fbc9df79903c4b6c997e50f35..b82692f65fa6912a8ec63019cf59a7918dc20d8b 100644
--- a/components/arc/common/auth.mojom
+++ b/components/arc/common/auth.mojom
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-// Next MinVersion: 9
+// Next MinVersion: 10
module arc.mojom;
@@ -66,6 +66,28 @@ enum ArcSignInFailureReason {
[MinVersion=8] NO_NETWORK_CONNECTION = 15,
};
+// These values describe account check status.
+[Extensible]
+enum AccountCheckStatus {
+ // Checked account exists and is up to date.
+ [MinVersion=9] UP_TO_DATE = 0,
+
+ // Checked account is new in the system.
+ [MinVersion=9] NEW = 1,
+
+ // Checked account exists but need to be reauthorized.
+ [MinVersion=9] NEED_REAUTH = 2,
+
+ // Checked account exists but it is in unknown state.
+ [MinVersion=9] UNKNOWN = 3,
+
+ // Account check has failed.
+ [MinVersion=9] CHECK_FAILED = 4,
+
+ // NOTE: If you add any entries to this enum, you must also update the
+ // corresponding UMA callsite in Chrome arc::UpdateAuthAccountCheckStatus.
+};
+
// These values describe the type of the Chrome account to provision.
[Extensible]
enum ChromeAccountType {
@@ -97,10 +119,16 @@ enum MetricsType {
// Duration of waiting for sign-in completed in milliseconds.
SIGNIN_TIME_MILLISECONDS = 3,
+
+ // Duration of checking the status of account.
+ [MinVersion=9] ACCOUNT_CHECK_MILLISECONDS = 4,
};
// The necessary information for Android to sign in and provision itself.
struct AccountInfo {
+ // Name of account, used to map to existing Android account.
+ [MinVersion=9] string? account_name@4;
+
// The authorization code that can be used in Android to sign in when
// account_type is USER_ACCOUNT or ROBOT_ACCOUNT. If it is null in these
// cases, sign-in will be skipped.
@@ -118,7 +146,7 @@ struct AccountInfo {
bool is_managed@2;
};
-// Next Method ID: 9.
+// Next Method ID: 10.
interface AuthHost {
// Notifies Chrome that the sign-in is completed successfully.
[MinVersion=2] OnSignInComplete@2();
@@ -132,6 +160,9 @@ interface AuthHost {
// Reports metrics to Chrome to be recorded in UMA.
[MinVersion=7] ReportMetrics@8(MetricsType metrics_type, int32 value);
+ // Reports result of account check.
+ [MinVersion=9] ReportAccountCheckStatus@9(AccountCheckStatus status);
+
// Deprecated methods:
// Returns an authorization code, which can be used to sign in.
« no previous file with comments | « chrome/browser/chromeos/arc/arc_optin_uma.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698