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

components/signin/core/browser/account_reconcilor.h

Index: components/signin/core/browser/account_reconcilor.h
diff --git a/components/signin/core/browser/account_reconcilor.h b/components/signin/core/browser/account_reconcilor.h
index 018dfd24be1a03e17559312274e39ae0e17db7c9..7083a653146a0eaa3d2553d51f5aa4e8aa0cc9d0 100644
--- a/components/signin/core/browser/account_reconcilor.h
+++ b/components/signin/core/browser/account_reconcilor.h
@@ -230,6 +230,11 @@ class AccountReconcilor : public KeyedService,
   // 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.