non-new-profile-management creates a "no-op" style account_reconcilor,

components/signin/core/browser/account_reconcilor.h

 // Copyright 2014 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.
 #ifndef COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_RECONCILOR_H_
 #define COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_RECONCILOR_H_
 
 #include <deque>
 #include <functional>
 #include <set>
 #include <string>
@@ skipping 212 matching lines @@
   // this profile.
   bool is_reconcile_started_;
 
   // Used during reconcile action.
   // These members are used used to validate the gaia cookie.  |gaia_accounts_|
   // holds the state of google accounts in the gaia cookie.  Each element is
   // a pair that holds the email address of the account and a boolean that
   // indicates whether the account is valid or not.  The accounts in the vector
   // are ordered the in same way as the gaia cookie.
   bool are_gaia_accounts_set_;
+
+  // Set noop_only mode of the account reconciler should do all the tracking

[noms (inactive), 2014/05/07 14:57:24]

I don't think I understand this comment very well. Would something like "If
|noop_only_mode_| is true, then the account reconciler will do all the
tracking..." ?

I also think the variable name is a little confusing; no-op without the dash
looks weird, and no-op-only sounds like a double negation :(
How about something like "skip_reconciliation" or "no_reconciliation_needed"
instead?

[Roger Tawa OOO till Jul 10th, 2014/05/07 16:14:45]

The mirror promo may change the value returned by IsNewProfileManagement() at
runtime.  So better not to cache this at all and just call the function each
time we need it.

Note that the value can only change from false->true.  True->false requires a
browser restart, so I think we are covered in all cases.

[Mike Lerman, 2014/05/08 15:17:50]

Ok, so since the value will not be cached this variable is removed.

+  // and statistical gathering that would be performed during reconciliation,
+  // but not perform any account, token or cookie merges.
+  bool noop_only_mode_;
   std::vector<std::pair<std::string, bool> > gaia_accounts_;
 
   // Used during reconcile action.
   // These members are used to validate the tokens in OAuth2TokenService.
   std::string primary_account_;
   std::vector<std::string> chrome_accounts_;
   scoped_ptr<OAuth2TokenService::Request>* requests_;
   ScopedVector<UserIdFetcher> user_id_fetchers_;
   ScopedVector<SigninOAuthHelper> refresh_token_fetchers_;
   EmailSet valid_chrome_accounts_;
   EmailSet invalid_chrome_accounts_;
   std::vector<std::string> add_to_cookie_;
   std::vector<std::pair<std::string, int> > add_to_chrome_;
 
   std::deque<GetAccountsFromCookieCallback> get_gaia_accounts_callbacks_;
 
   DISALLOW_COPY_AND_ASSIGN(AccountReconcilor);
 };
 
 #endif  // COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_RECONCILOR_H_