Disable UI flags for non-desktop OSes

chrome/browser/android/signin/signin_manager_android.cc

Index: chrome/browser/android/signin/signin_manager_android.cc
diff --git a/chrome/browser/android/signin/signin_manager_android.cc b/chrome/browser/android/signin/signin_manager_android.cc
index 34ebb4d058754384fda48ae72038d21fd066b8aa..5bf248ab5ef42db7bdef695679ef9f075ced219d 100644
--- a/chrome/browser/android/signin/signin_manager_android.cc
+++ b/chrome/browser/android/signin/signin_manager_android.cc
@@ -232,7 +232,7 @@ void SigninManagerAndroid::MergeSessionCompleted(
 void SigninManagerAndroid::LogInSignedInUser(JNIEnv* env, jobject obj) {
   SigninManagerBase* signin_manager =
       SigninManagerFactory::GetForProfile(profile_);
-  if (switches::IsNewProfileManagement()) {
+  if (switches::IsEnableAccountConsistency()) {
     // New Mirror code path that just fires the events and let the
     // Account Reconcilor handles everything.
     AndroidProfileOAuth2TokenService* token_service =
@@ -286,7 +286,7 @@ static jboolean ShouldLoadPolicyForUser(JNIEnv* env,
 }
 
 static jboolean IsNewProfileManagementEnabled(JNIEnv* env, jclass clazz) {
-  return switches::IsNewProfileManagement();
+  return switches::IsEnableAccountConsistency();

[aruslan, 2014/08/18 21:25:21]

This is a deadly way of dealing with flags.
It is really hard to tell if there is a difference between
"IsNewProfileManagementEnabled" and "IsNewProfileManagement".

[Mike Lerman, 2014/08/19 14:59:07]

I don't see a IsNewProfileManagementEnabled flag, except for this method here in
signin_manager_android. I assume this is the method that controls/passes the
relevant flag through to the Java-side of Clank? It should (I imagine) be called
IsAccountConsistencyEnabled, since NewProfileManagement has no meaning on
Android.

 }
 
 // static