Keeps the opt-in IME menu in lock screen.

chrome/browser/chromeos/login/session/user_session_manager.cc

Index: chrome/browser/chromeos/login/session/user_session_manager.cc
diff --git a/chrome/browser/chromeos/login/session/user_session_manager.cc b/chrome/browser/chromeos/login/session/user_session_manager.cc
index 8ece9ff1b661c9c13483348e7f1c03309773acb6..e8052091dbccee4d95ad24f325cb517a0fd85fbd 100644
--- a/chrome/browser/chromeos/login/session/user_session_manager.cc
+++ b/chrome/browser/chromeos/login/session/user_session_manager.cc
@@ -1655,6 +1655,7 @@ void UserSessionManager::ActiveUserChanged(
       input_method::InputMethodManager::Get();
   manager->SetState(
       GetDefaultIMEState(ProfileHelper::Get()->GetProfileByUser(active_user)));
+  manager->ImeMenuActivationChanged(manager->GetImeMenuActivation(profile));

[Shu Chen, 2016/08/27 03:11:02]

This Set(Get(...)) pattern is kinda confusing.

I'm thinking to add the ImeMenuActivation state in StateImpl.

WDYT?

[Azure Wei, 2016/08/27 06:55:30]

The saved menu_activated_ in StateImpl will also be casted into false here 
(cast from State to StateImpl). But if we put menu_activated_ in
InputMethodManager::State, there are many other problems:
InputMethodManager::SetState() will be called in many places (login/adding
user/lock screen/switching user), but we don't really want to set the menu
activation state in those places. We only want to set/change menu activation
state when switching users. Lock/Adding user screen should keep the same with
the current user. 
I tried to put menu_activaed_ in InputMethodManager::State or
InputMethodManagerImpl::StateImpl, there are many bugs under these conditions. 

So, do you mind we keeping the current way?
Or would you prefer adding a new event like ActiveUserChanged() in
InputMethodManager? 

[Azure Wei, 2016/08/29 02:34:40]

Per offline discussion, saved menu_activated state in StateImple. Thus we don't
need menu_activated_for_profile_in InputMethodManagerImpl any more.

 }
 
 scoped_refptr<input_method::InputMethodManager::State>