Improve error messaging with shared module import failures

chrome/browser/extensions/webstore_installer.cc

Index: chrome/browser/extensions/webstore_installer.cc
diff --git a/chrome/browser/extensions/webstore_installer.cc b/chrome/browser/extensions/webstore_installer.cc
index a378a494fc3029054aff03dddaa484392eb1a25a..454197c086222e504a1b29e1bcf696b31e7864ca 100644
--- a/chrome/browser/extensions/webstore_installer.cc
+++ b/chrome/browser/extensions/webstore_installer.cc
@@ -309,17 +309,11 @@ void WebstoreInstaller::Start() {
   ExtensionService* extension_service =
     ExtensionSystem::Get(profile_)->extension_service();
   if (approval_.get() && approval_->dummy_extension.get()) {
-    SharedModuleService::ImportStatus status =
-        extension_service->shared_module_service()->CheckImports(
-            approval_->dummy_extension.get(),
-            &pending_modules_,
-            &pending_modules_);
-    // For this case, it is because some imports are not shared modules.
-    if (status == SharedModuleService::IMPORT_STATUS_UNRECOVERABLE) {
-      ReportFailure(kDependencyNotSharedModuleError,
-          FAILURE_REASON_DEPENDENCY_NOT_SHARED_MODULE);
-      return;
-    }
+    extension_service->shared_module_service()->CheckImports(
+        approval_->dummy_extension.get(), &pending_modules_, &pending_modules_);
+    // Do not check the return value of CheckImports, the CRX installer
+    // will report appropriate error messages and fail to install if there
+    // is an import error.
   }
 
   // Add the extension main module into the list.