Basic multi-module support

chrome/browser/extensions/crx_installer.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/crx_installer.h"
 
 #include <map>
 #include <set>
 
 #include "base/bind.h"
@@ skipping 24 matching lines @@
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/web_applications/web_app.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/extensions/extension_file_util.h"
 #include "chrome/common/extensions/extension_icon_set.h"
 #include "chrome/common/extensions/feature_switch.h"
 #include "chrome/common/extensions/manifest.h"
 #include "chrome/common/extensions/manifest_handlers/icons_handler.h"
+#include "chrome/common/extensions/manifest_handlers/shared_module_info.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/resource_dispatcher_host.h"
 #include "content/public/browser/user_metrics.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 
 #if defined(ENABLE_MANAGED_USERS)
 #include "chrome/browser/managed_mode/managed_user_service.h"
 #include "chrome/browser/managed_mode/managed_user_service_factory.h"
 #endif
 
 using content::BrowserThread;
 using content::UserMetricsAction;
+using extensions::SharedModuleInfo;
 
 namespace extensions {
 
 namespace {
 
 // Used in histograms; do not change order.
 enum OffStoreInstallDecision {
   OnStoreInstall,
   OffStoreInstallAllowed,
   OffStoreInstallDisallowed,
@@ skipping 320 matching lines @@
 
   if (client_) {
     IconsInfo::DecodeIcon(extension_.get(),
                           extension_misc::EXTENSION_ICON_LARGE,
                           ExtensionIconSet::MATCH_BIGGER,
                           &install_icon_);
   }
 
   if (!BrowserThread::PostTask(
         BrowserThread::UI, FROM_HERE,
-        base::Bind(&CrxInstaller::CheckRequirements, this)))
+        base::Bind(&CrxInstaller::CheckImportsAndRequirements, this)))
     NOTREACHED();
 }
 
-void CrxInstaller::CheckRequirements() {
+void CrxInstaller::CheckImportsAndRequirements() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   if (!frontend_weak_.get() || frontend_weak_->browser_terminating())
     return;
+
+  if (SharedModuleInfo::ImportsModules(extension_)) {

[Matt Perry, 2013/04/25 20:16:31]

What if you uninstall a shared module that something depends on?

[elijahtaylor1, 2013/04/25 21:01:28]

In the final version we had talked about making shared modules not visible from
the normal chrome://extensions UI and thus uninstallable by the user.  The only
way a shared module would be uninstalled is when it is no longer referenced by
any other installed extension.


It's kind of dependent on the ability to auto-download/install shared modules on
demand to complete the entire seamless experience.  Let me know if you feel
strongly about leaving this dangling piece of functionality out there for now.

[Matt Perry, 2013/04/25 21:07:23]

Oh yeah, that's right. Makes sense. Leave as-is.

+    const std::vector<SharedModuleInfo::ImportInfo>& imports =
+        SharedModuleInfo::GetImports(extension_);
+    std::vector<SharedModuleInfo::ImportInfo>::const_iterator i;
+    for (i = imports.begin(); i != imports.end(); ++i) {
+      Version version_required(i->minimum_version);
+      const Extension* imported_module =
+          frontend_weak_->GetExtensionById(i->extension_id, true);
+      if (!imported_module ||
+          imported_module->version()->CompareTo(version_required) < 0) {
+        ReportFailureFromUIThread(
+            CrxInstallerError(l10n_util::GetStringFUTF16(
+                IDS_EXTENSION_INSTALL_DEPENDENCY_NOT_FOUND,
+                ASCIIToUTF16(i->extension_id),
+                ASCIIToUTF16(i->minimum_version))));
+        return;
+      }
+      if (!SharedModuleInfo::IsSharedModule(imported_module)) {
+        ReportFailureFromUIThread(
+            CrxInstallerError(l10n_util::GetStringFUTF16(
+                IDS_EXTENSION_INSTALL_DEPENDENCY_NOT_SHARED_MODULE,
+                ASCIIToUTF16(i->extension_id))));
+        return;
+      }
+    }
+  }
   AddRef();  // Balanced in OnRequirementsChecked().
   requirements_checker_->Check(extension_,
                                base::Bind(&CrxInstaller::OnRequirementsChecked,
                                           this));
 }
 
 void CrxInstaller::OnRequirementsChecked(
     std::vector<std::string> requirement_errors) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  Release();  // Balanced in CheckRequirements().
+  Release();  // Balanced in CheckImportsAndRequirements().
   if (!requirement_errors.empty()) {
     if (error_on_unsupported_requirements_) {
       ReportFailureFromUIThread(CrxInstallerError(
           UTF8ToUTF16(JoinString(requirement_errors, ' '))));
       return;
     }
     has_requirement_errors_ = true;
   }
 
 #if defined(ENABLE_MANAGED_USERS) && !defined(OS_CHROMEOS)
@@ skipping 382 matching lines @@
   if (!prefs->DidExtensionEscalatePermissions(extension_->id()))
     return;
 
   if (client_) {
     AddRef();  // Balanced in InstallUIProceed() and InstallUIAbort().
     client_->ConfirmReEnable(this, extension_.get());
   }
 }
 
 }  // namespace extensions