arc: Skip GMS Sign-In in case Android is already signed-in.

components/arc/common/auth.mojom

 // Copyright 2015 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.
 
-// Next MinVersion: 9
+// Next MinVersion: 10
 
 module arc.mojom;
 
 // These values describe failure reason of sign-in.
 [Extensible]
 enum ArcSignInFailureReason {
   // Negative values are reserved for internal use.
   // The values are shuffled to keep the backward compatibility and don't match
   // ProvisioningResult in arc_optin_uma.h
   // Next value: 16.
@@ skipping 43 matching lines @@
 
   // Cloud provisioning errors.
   CLOUD_PROVISION_FLOW_FAILED = 5,
   CLOUD_PROVISION_FLOW_TIMEOUT = 13,
   CLOUD_PROVISION_FLOW_INTERNAL_ERROR = 14,
 
   // Network connection is unavailable.
   [MinVersion=8] NO_NETWORK_CONNECTION = 15,
 };
 
+// These values describe account check status. In case extending this enum you
+// have to update UMA reporting in Chrome. See

[Ilya Sherman, 2017/03/27 19:03:28]

nit: I'd phrase the second sentence as something like "NOTE: If you add any
entries to this enum, you must also update the corresponding UMA callsite in
Chrome."  I'd also suggest moving that comment to the end of the enum, rather
than the top, though I find that readers are likely to miss this sort of comment
in either case.

[khmel, 2017/03/27 19:29:29]

Makes sense, thanks!

+// arc::UpdateAuthAccountCheckStatus.
+[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,
+};
+
 // These values describe the type of the Chrome account to provision.
 [Extensible]
 enum ChromeAccountType {
   // Next value: 4
   UNKNOWN = 0,
 
   // Chrome login account type is a user account.
   USER_ACCOUNT = 1,
 
   // Chrome login account type is a robot (service) account.
@@ skipping 11 matching lines @@
 
   // Number of attempts to wait for Checkin task completed. 0 indicates that
   // Checkin task was already completed prior to GMS SignIn.
   CHECKIN_ATTEMPTS = 1,
 
   // Duration of waiting for Checkin task completed in milliseconds.
   CHECKIN_TIME_MILLISECONDS = 2,
 
   // 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.
   string? auth_code@0;
 
   // If account_type is ACTIVE_DIRECTORY_ACCOUNT, this contains an enrollment
   // token for a Managed Google Play account. If it is null in this case,
   // sign-in will be skipped.
   [MinVersion=6] string? enrollment_token@3;
 
   // The type of Chrome account to provision.
   ChromeAccountType account_type@1;
 
   // Whether the account is managed from Chrome OS.
   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();
   // Notifies Chrome that the sign-in fails to complete and provides failure
   // reason.
   [MinVersion=2] OnSignInFailed@3(ArcSignInFailureReason reason);
   // Asynchronously requests an authorization code, as well as the account
   // information.
   [MinVersion=5] RequestAccountInfo@7();
 
   // 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.
   GetAuthCodeDeprecated0@0() => (string auth_code);
   // Returns an authorization code in case is_enforced is set, which can be
   // used to sign in. By default the ChromeAccountType is USER_ACCOUNT.
   [MinVersion=1] GetAuthCodeDeprecated@1() => (string auth_code,
                                                bool is_enforced);
   // Returns an authorization code and its ChromeAccountType in case
   // is_enforced is set, which can be used to sign in.
   [MinVersion=4] GetAuthCodeAndAccountTypeDeprecated@5() => (
           string auth_code, bool is_enforced, ChromeAccountType account_type);
   // Gets whether the account is managed from Chrome OS.
   [MinVersion=3] GetIsAccountManagedDeprecated@4() => (bool is_managed);
 };
 
 // Next Method ID: 2
 interface AuthInstance {
   // Establishes full-duplex communication with the host.
   Init@0(AuthHost host_ptr);
 
   // Callback from RequestAccountInfo. This cannot be a normal callback since
   // the result can sometimes take a few minutes in some cases (Kiosk mode),
   // and in other cases there is a UI that the user is shown and they can take
   // an arbitrarily long amount of time to interact with.
   // If |account_info| is null, it implies that there was an error in the
   // process.
   [MinVersion=5] OnAccountInfoReady(AccountInfo? account_info);
 };