Show alert failure for reloading unpacked extensions with bad manifest

chrome/browser/extensions/unpacked_installer.cc

 // Copyright (c) 2013 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/unpacked_installer.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/file_util.h"
@@ skipping 278 matching lines @@
   }
 
   BrowserThread::PostTask(
       BrowserThread::UI,
       FROM_HERE,
       base::Bind(&UnpackedInstaller::ShowInstallPrompt, this));
 }
 
 void UnpackedInstaller::ReportExtensionLoadError(const std::string &error) {
   CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  if (!on_failure_callback_.is_null())
-    on_failure_callback_.Run(extension_path_, error);
 
   if (service_weak_.get()) {
     ExtensionErrorReporter::GetInstance()->ReportLoadError(
         extension_path_,
         error,
         service_weak_->profile(),
         be_noisy_on_failure_);
   }
 }
 
+

[Devlin, 2014/06/27 22:31:08]

nit: remove extraneous newline.

[gpdavis, 2014/06/28 02:31:17]

Done.

 void UnpackedInstaller::ConfirmInstall() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   base::string16 error = installer_.CheckManagementPolicy();
   if (!error.empty()) {
     ReportExtensionLoadError(base::UTF16ToUTF8(error));
     return;
   }
 
   PermissionsUpdater perms_updater(service_weak_->profile());
   perms_updater.GrantActivePermissions(installer_.extension().get());
 
   service_weak_->OnExtensionInstalled(installer_.extension().get(),
                                       syncer::StringOrdinal(),
                                       kInstallFlagInstallImmediately);
 }
 
 }  // namespace extensions