Check IME state in ImeController

chrome/browser/extensions/extension_error_reporter.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/extension_error_reporter.h"
 
 #include "build/build_config.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 59 matching lines @@
   CHECK(!ui_task_runner_ || ui_task_runner_->BelongsToCurrentThread())
       << "ReportError can only be called from the UI thread.";
 
   errors_.push_back(message);
 
   // TODO(aa): Print the error message out somewhere better. I think we are
   // going to need some sort of 'extension inspector'.
   LOG(WARNING) << "Extension error: " << message;
 
   if (enable_noisy_errors_ && be_noisy) {
+    if (should_postpone_messages_) {
+      postponed_messages_.push_back(message);
+      return;
+    }
     chrome::ShowWarningMessageBox(
         NULL, l10n_util::GetStringUTF16(IDS_EXTENSIONS_LOAD_ERROR_HEADING),
         message);
   }
 }
 
+void ExtensionErrorReporter::ShowPostponedMessages() {
+  should_postpone_messages_ = false;
+  for (base::string16 message : postponed_messages_) {
+    chrome::ShowWarningMessageBox(

[tbarzic, 2017/04/25 22:25:55]

it would be nice to condense multiple messages into a single warning, but I
guess even before this we'd show a message box per error, so it's not really
making things worse.

+        NULL, l10n_util::GetStringUTF16(IDS_EXTENSIONS_LOAD_ERROR_HEADING),
+        message);
+  }
+  postponed_messages_.clear();
+}
+
 const std::vector<base::string16>* ExtensionErrorReporter::GetErrors() {
   return &errors_;
 }
 
 void ExtensionErrorReporter::ClearErrors() {
   errors_.clear();
 }
 
 void ExtensionErrorReporter::AddObserver(Observer* observer) {
   observers_.AddObserver(observer);
 }
 
 void ExtensionErrorReporter::RemoveObserver(Observer* observer) {
   observers_.RemoveObserver(observer);
 }