Show alert failure for reloading unpacked extensions with bad manifest

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 "chrome/browser/extensions/extension_error_reporter_observer.h"
 
 #include "build/build_config.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 31 matching lines @@
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_EXTENSION_LOAD_ERROR,
       content::Source<Profile>(profile),
       content::Details<const std::string>(&error));
 
   std::string path_str = base::UTF16ToUTF8(extension_path.LossyDisplayName());
   base::string16 message = base::UTF8ToUTF16(
       base::StringPrintf("Could not load extension from '%s'. %s",
                          path_str.c_str(),
                          error.c_str()));
-  ReportError(message, be_noisy);
+  // ReportError(message, be_noisy);
+  ReportError(message, false);
+  TriggerOnLoadFailure(extension_path, error);
 }
 
 void ExtensionErrorReporter::ReportError(const base::string16& message,
                                          bool be_noisy) {
+  DLOG(ERROR) << "----ReportError";
   // NOTE: There won't be a ui_loop_ in the unit test environment.
   if (ui_loop_) {
     CHECK(base::MessageLoop::current() == ui_loop_)
         << "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) {
     chrome::ShowMessageBox(NULL,
                            base::ASCIIToUTF16("Extension error"),
                            message,
                            chrome::MESSAGE_BOX_TYPE_WARNING);
   }
 }
 
 const std::vector<base::string16>* ExtensionErrorReporter::GetErrors() {
   return &errors_;
 }
 
 void ExtensionErrorReporter::ClearErrors() {
   errors_.clear();
 }
+
+void ExtensionErrorReporter::AddObserver(
+    ExtensionErrorReporterObserver* observer) {
+  observers_.AddObserver(observer);
+}
+
+void ExtensionErrorReporter::RemoveObserver(
+    ExtensionErrorReporterObserver* observer) {
+  observers_.RemoveObserver(observer);
+}
+
+void ExtensionErrorReporter::TriggerOnLoadFailure(
+    const base::FilePath& extension_path,
+    const std::string& error) {
+  DLOG(ERROR) << "---TriggerOnLoadFailure";
+  FOR_EACH_OBSERVER(ExtensionErrorReporterObserver,
+                    observers_,
+                    OnLoadFailure(extension_path, error));
+}