Change semantics of newProfileManagement() to accountConsistency()

chrome/browser/signin/signin_tracker_factory.cc

Index: chrome/browser/signin/signin_tracker_factory.cc
diff --git a/chrome/browser/signin/signin_tracker_factory.cc b/chrome/browser/signin/signin_tracker_factory.cc
index 5b1febc92885436dfe23f7f933aa6badd9700e37..8cb37d18f291b81d04fa4640ad8deb334d9175cd 100644
--- a/chrome/browser/signin/signin_tracker_factory.cc
+++ b/chrome/browser/signin/signin_tracker_factory.cc
@@ -19,7 +19,7 @@ scoped_ptr<SigninTracker> SigninTrackerFactory::CreateForProfile(
     SigninTracker::Observer* observer) {
   // Determine whether to use the AccountReconcilor.
   AccountReconcilor* account_reconcilor = NULL;
-  if (!switches::IsEnableWebBasedSignin() && switches::IsNewProfileManagement())
+  if (!switches::IsEnableWebBasedSignin())

[guohui, 2014/07/07 21:06:57]

so we want to always create an account reconcilor unless web signin is enabled?
why?

[Mike Lerman, 2014/07/08 12:30:48]

I removed the check for the NewProfileManagement flag because that is now done
within the AccountReconcilor. 
I didn't touch WebBasedSignin because I'm not aware of anything which would
change the need for it to be here.

     account_reconcilor = AccountReconcilorFactory::GetForProfile(profile);
 
   return scoped_ptr<SigninTracker>(new SigninTracker(