[Signin] Create Multilogin::Mode variable to prepare for logging.

Set mode to UPDATE for Mirror and log it. Needed for CL on logging which has a higher priority than the one where mode will actually be used. Use VLOG(1) to make variable and used somewhere.

Change-Id: I0eb3def674b510a5a5ca433f62540a027fa1cc25
Reviewed-on: https://chromium-review.googlesource.com/1255823
Commit-Queue: Valeriya Sinevich <valeriyas@google.com>
Reviewed-by: David Roger <droger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595829}

components/signin/core/browser/account_reconcilor.cc

@@ -432,6 +432,12 @@ void AccountReconcilor::FinishReconcileWithMultiloginEndpoint(
     std::vector<gaia::ListedAccount>&& gaia_accounts) {
   DCHECK(base::FeatureList::IsEnabled(kUseMultiloginEndpoint));
 
+  signin::MultiloginMode mode;
+  if (delegate_->ShouldUpdateAccountsOrderInCookies())
+    mode = signin::MultiloginMode::MULTILOGIN_UPDATE_COOKIE_ACCOUNTS_ORDER;
+  else
+    mode = signin::MultiloginMode::MULTILOGIN_PRESERVE_COOKIE_ACCOUNTS_ORDER;
+
   std::vector<std::string> accounts_to_send;
   if (delegate_->ReorderChromeAccountsForReconcileIfNeeded(
           chrome_accounts, primary_account, gaia_accounts, &accounts_to_send)) {
@@ -439,7 +445,13 @@ void AccountReconcilor::FinishReconcileWithMultiloginEndpoint(
   } else {
     OnSetAccountsInCookieCompleted(GoogleServiceAuthError::AuthErrorNone());
   }
-  // TODO(valeriyas): Log operation here.
+
+  VLOG(1) << "AccountReconcilor::FinishReconcileWithMultiloginEndpoint "
+          << "in mode: "
+          << (mode == signin::MultiloginMode::
+                          MULTILOGIN_UPDATE_COOKIE_ACCOUNTS_ORDER
+                  ? "UPDATE"
+                  : "PRESERVE");
   // TODO (valeriyas): Write correct first gaia account to cache for desktop.
   if (!is_reconcile_started_)
     delegate_->OnReconcileFinished(primary_account, reconcile_is_noop_);

components/signin/core/browser/account_reconcilor_delegate.cc

@@ -44,6 +44,10 @@ bool AccountReconcilorDelegate::ReorderChromeAccountsForReconcileIfNeeded(
   return false;
 }
 
+bool AccountReconcilorDelegate::ShouldUpdateAccountsOrderInCookies() const {
+  return true;
+}
+
 AccountReconcilorDelegate::RevokeTokenOption
 AccountReconcilorDelegate::ShouldRevokeSecondaryTokensBeforeReconcile(
     const std::vector<gaia::ListedAccount>& gaia_accounts) {

components/signin/core/browser/account_reconcilor_delegate.h

@@ -9,6 +9,7 @@
 #include <vector>
 
 #include "base/time/time.h"
+#include "google_apis/gaia/gaia_auth_fetcher.h"
 #include "google_apis/gaia/gaia_auth_util.h"
 #include "google_apis/gaia/google_service_auth_error.h"
 
@@ -71,6 +72,10 @@ class AccountReconcilorDelegate {
       const std::vector<gaia::ListedAccount>& gaia_accounts,
       std::vector<std::string>* accounts_to_send) const;
 
+  // Returns true if it is allowed to change the order of the gaia accounts
+  // (e.g. on mobile or on stratup). Default is true.
+  virtual bool ShouldUpdateAccountsOrderInCookies() const;
+
   // Returns whether secondary accounts should be revoked at the beginning of
   // the reconcile.
   virtual RevokeTokenOption ShouldRevokeSecondaryTokensBeforeReconcile(

google_apis/gaia/gaia_auth_fetcher.h

@@ -33,6 +33,15 @@
 
 class GaiaAuthFetcherTest;
 
+namespace signin {
+// Mode determining whether Gaia can change the order of the accounts in
+// cookies.
+enum class MultiloginMode {
+  MULTILOGIN_UPDATE_COOKIE_ACCOUNTS_ORDER = 0,
+  MULTILOGIN_PRESERVE_COOKIE_ACCOUNTS_ORDER
+};
+}  // namespace signin
+
 namespace network {
 class SimpleURLLoader;
 class SharedURLLoaderFactory;