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..88d16ef4ee0feb8058981c959469450cbad1f7c4 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,13 @@ void ChromeBrowserMainPartsChromeos::PostProfileInit() {
 
   manager->SetState(session_manager->GetDefaultIMEState(profile()));
 
+  // Show postponed extension error messages (if any) if it is the active user
+  // profile. Because extension errors are always reported in context of active
+  // user, it's safe to display the message boxes after input methods of the
+  // active user is initialized.
+  if (ProfileHelper::Get()->GetUserByProfile(profile())->is_active())

[xiyuan, 2017/04/24 18:21:51]

Most of the time (i.e. NOT the crash-n-restart case), profile() is the sign-in
profile. There is no user associated with it. So, this line would probably
crash.

[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);