Check IME state in ImeController

chrome/browser/chromeos/chrome_browser_main_chromeos.cc

Index: chrome/browser/chromeos/chrome_browser_main_chromeos.cc
diff --git a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
index 87eafaca02864b1b556f87c807bb133995a9c5f0..61a53d8c57e7ca708b45ee21a22e51002d3fa3ab 100644
--- a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
+++ b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
@@ -90,6 +90,7 @@
 #include "chrome/browser/chromeos/ui/low_disk_notification.h"
 #include "chrome/browser/chromeos/upgrade_detector_chromeos.h"
 #include "chrome/browser/defaults.h"
+#include "chrome/browser/extensions/extension_error_reporter.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/net/chrome_network_delegate.h"
 #include "chrome/browser/profiles/profile.h"
@@ -748,6 +749,11 @@ void ChromeBrowserMainPartsChromeos::PostProfileInit() {
 
   manager->SetState(session_manager->GetDefaultIMEState(profile()));
 
+  // Safe to show extension error message boxes (if any) now as input methods
+  // of the active user is initialized.
+  if (ProfileHelper::Get()->GetUserByProfile(profile())->is_active())

[Alexander Alekseev, 2017/04/24 11:43:27]

Maybe we should add more details to this comment.
---------------
This works because ExtensionErrors are always reported in context of active
user, even if they refer to another profile.

So if we are in the process of restart after crash in multiprofile mode, primary
user profile will be initialized first, so it is safe to display error balloons
after first user pfofile is initialized.

[Wenzhao (Colin) Zang, 2017/04/25 22:06:32]

Done.

+    ExtensionErrorReporter::GetInstance()->ShowPostponedMessages();
+
   bool is_running_test = parameters().ui_task != nullptr;
   g_browser_process->platform_part()->session_manager()->Initialize(
       parsed_command_line(), profile(), is_running_test);