Don't switch browser locale on secondary user login

chrome/browser/chromeos/login/login_utils.cc

Index: chrome/browser/chromeos/login/login_utils.cc
diff --git a/chrome/browser/chromeos/login/login_utils.cc b/chrome/browser/chromeos/login/login_utils.cc
index 552b5af58e5e47496b482820f9352f3597301649..aa2e584e3090ca8f3bfca23d93edc8246a2aaed7 100644
--- a/chrome/browser/chromeos/login/login_utils.cc
+++ b/chrome/browser/chromeos/login/login_utils.cc
@@ -378,7 +378,9 @@ void LoginUtilsImpl::InitProfilePreferences(Profile* user_profile,
     google_services_username.SetValue(user ? user->display_email() : email);
   }
 
-  RespectLocalePreference(user_profile);
+  // For multi-profile case don't apply profile local because it is not safe.
+  if (UserManager::Get()->GetLoggedInUsers().size() == 1)

[Mr4D (OOO till 08-26), 2013/09/13 23:41:36]

Don't know how this plays into it, but while debugging I have seen that the
login screen is also trying to set the user input locale
(signin_screen_handler.cc(1044)). I guess you should check that out quickly as
well).

Oh and: I am not sure that you can assume that the UserManager is already there.
You might want to check if you have to look at UserManager::IsInitialized().

[Dmitry Polukhin, 2013/09/14 00:28:16]

Good catch! It is place where we can switch input methods (not locale). Fixed.
It must be initialized already because UserManaged initiates profile creation.
But after crash not all user could be loaded, in this case we do want to apply
first user settings (it should be first logged in user) and skip other user
settings.

+    RespectLocalePreference(user_profile);
 }
 
 void LoginUtilsImpl::InitSessionRestoreStrategy() {