The init of Preferences instance should affect the singleton...

The init of Preferences instance should affect the singleton InputMethodManager state only when the preferences belongs to the active user.

Bug: 841112
Change-Id: I7ed3cb2cbd8c0a27cab869f336c93c1a5becefa6
Reviewed-on: https://chromium-review.googlesource.com/1125646
Commit-Queue: Shu Chen <shuchen@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#573595}

chrome/browser/chromeos/preferences.cc

@@ -567,6 +567,10 @@ void Preferences::Init(Profile* profile, const user_manager::User* user) {
   // SetState() is modifying |current_input_method_| (via
   // PersistUserInputMethod() ). This way SetState() here may be called only
   // after ApplyPreferences().
+  // As InputMethodManager only holds the active state for the active user,
+  // SetState() is only called if the preferences belongs to the active user.
+  // See https://crbug.com/841112.
+  if (user->is_active())
     input_method_manager_->SetState(ime_state_);
 
   input_method_syncer_.reset(