Add minimum version to extension management

chrome/browser/extensions/extension_service.cc

Index: chrome/browser/extensions/extension_service.cc
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc
index 283f42a1fc3059b09f965b156b5455ac1dbfa62c..04d78c41d877d07d101ad90f1a52917569c06a2b 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -853,6 +853,7 @@ void ExtensionService::DisableExtension(
   // preferences.
   if (extension &&
       disable_reason != Extension::DISABLE_RELOAD &&
+      disable_reason != Extension::DISABLE_UPDATE_REQUIRED_BY_POLICY &&
       !system_->management_policy()->UserMayModifySettings(extension, NULL) &&
       extension->location() != Manifest::EXTERNAL_COMPONENT) {
     return;
@@ -1111,27 +1112,71 @@ base::SequencedTaskRunner* ExtensionService::GetFileTaskRunner() {
 void ExtensionService::CheckManagementPolicy() {
   std::vector<std::string> to_unload;
   std::map<std::string, Extension::DisableReason> to_disable;
+  std::vector<std::string> to_enable;
 
   // Loop through the extensions list, finding extensions we need to unload or
   // disable.
-  const ExtensionSet& extensions = registry_->enabled_extensions();
-  for (ExtensionSet::const_iterator iter = extensions.begin();
-       iter != extensions.end(); ++iter) {
-    const Extension* extension = (iter->get());
-    if (!system_->management_policy()->UserMayLoad(extension, NULL))
+  for (scoped_refptr<const Extension> extension :
+       registry_->enabled_extensions()) {
+    if (!system_->management_policy()->UserMayLoad(extension.get(), nullptr))
       to_unload.push_back(extension->id());
     Extension::DisableReason disable_reason = Extension::DISABLE_NONE;
     if (system_->management_policy()->MustRemainDisabled(
-            extension, &disable_reason, NULL))
+            extension.get(), &disable_reason, nullptr))
       to_disable[extension->id()] = disable_reason;
   }
 
-  for (size_t i = 0; i < to_unload.size(); ++i)
-    UnloadExtension(to_unload[i], UnloadedExtensionInfo::REASON_DISABLE);
+  extensions::ExtensionManagement* management =
+      extensions::ExtensionManagementFactory::GetForBrowserContext(profile());
+
+  // Loop through the disabled extension list, find extensions to re-enable
+  // automatically.
+  for (scoped_refptr<const Extension> extension :
+       registry_->disabled_extensions()) {
+    // Find all disabled extensions disabled due to minimum version requirement,
+    // but now satisfying it.
+    if (management->CheckMinimumVersion(extension.get(), nullptr) &&
+        extension_prefs_->HasDisableReason(
+            extension->id(), Extension::DISABLE_UPDATE_REQUIRED_BY_POLICY)) {
+      if (extension_prefs_->GetDisableReasons(extension->id()) ==
+          Extension::DISABLE_UPDATE_REQUIRED_BY_POLICY) {
+          // We need to enable those disabled only due to minimum version
+          // requirement.
+          to_enable.push_back(extension->id());
+          extension_prefs_->ClearDisableReasons(extension->id());
+        } else {
+          // otherwise, there are other disable reasons, just remove ours.
+          extension_prefs_->RemoveDisableReason(
+              extension->id(), Extension::DISABLE_UPDATE_REQUIRED_BY_POLICY);
+        }
+    }
+  }
+
+  for (const std::string& id : to_unload)
+    UnloadExtension(id, UnloadedExtensionInfo::REASON_DISABLE);
 
   for (std::map<std::string, Extension::DisableReason>::const_iterator i =
            to_disable.begin(); i != to_disable.end(); ++i)
     DisableExtension(i->first, i->second);
+
+  for (const std::string& id : to_enable)

[Joao da Silva, 2014/11/18 14:18:56]

Actually, we can also use "const auto&" here. I think it's fine as is.

[binjin, 2014/11/18 14:40:09]

Acknowledged.

+    EnableExtension(id);
+
+  if (updater_.get()) {
+    // Find all extensions (including the ones that got disabled just now)
+    // disabled due to minimum version requirement from policy, and check for
+    // update.
+    extensions::ExtensionUpdater::CheckParams params;
+    for (scoped_refptr<const Extension> extension :
+         registry_->disabled_extensions()) {
+      if (extension_prefs_->HasDisableReason(
+              extension->id(), Extension::DISABLE_UPDATE_REQUIRED_BY_POLICY)) {
+        params.ids.push_back(extension->id());
+      }
+    }
+    if (!params.ids.empty())
+      updater_->CheckNow(params);
+  }
 }
 
 void ExtensionService::CheckForUpdatesSoon() {
@@ -1607,13 +1652,33 @@ void ExtensionService::OnExtensionInstalled(
   // Unsupported requirements overrides the management policy.
   if (install_flags & extensions::kInstallFlagHasRequirementErrors) {
     disable_reasons |= Extension::DISABLE_UNSUPPORTED_REQUIREMENT;
-  // If the extension was disabled because of unsupported requirements but
-  // now supports all requirements after an update and there are not other
-  // disable reasons, enable it.
   } else if (extension_prefs_->GetDisableReasons(id) ==
              Extension::DISABLE_UNSUPPORTED_REQUIREMENT) {
+    // The extension was disabled because of unsupported requirements but
+    // now supports all requirements after an update.
+    // If there are no other disable reasons, enable it.
     disable_reasons = Extension::DISABLE_NONE;
     extension_prefs_->ClearDisableReasons(id);
+  } else {
+    // Otherwise just remove the corresponding disable reason.
+    extension_prefs_->RemoveDisableReason(
+        id, Extension::DISABLE_UNSUPPORTED_REQUIREMENT);

[Joao da Silva, 2014/11/18 14:18:56]

|disable_reasons| needs to be updated after this call (also below)

[binjin, 2014/11/18 14:40:09]

Done.

+  }
+
+  // Check if the extension was disabled because of the minimum version
+  // requirements from enterprise policy, and satisfies it now.
+  if (extensions::ExtensionManagementFactory::GetForBrowserContext(profile())
+          ->CheckMinimumVersion(extension, nullptr)) {
+    if (extension_prefs_->GetDisableReasons(id) ==
+        Extension::DISABLE_UPDATE_REQUIRED_BY_POLICY) {
+      // There are no other disable reasons, re-enable the extension.
+      disable_reasons = Extension::DISABLE_NONE;
+      extension_prefs_->ClearDisableReasons(id);
+    } else {
+      // Otherwise just clear the corresponding disable reason.
+      extension_prefs_->RemoveDisableReason(
+          id, Extension::DISABLE_UPDATE_REQUIRED_BY_POLICY);

[Joao da Silva, 2014/11/18 14:18:56]

Update |disable_reasons| to remove DISABLE_UPDATE_REQUIRED_BY_POLICY, or just
get it again after removing this reason.

Note that |disable_reasons| will be checked again below.

[binjin, 2014/11/18 14:40:09]

Done.

+    }
   }
 
   if (install_flags & extensions::kInstallFlagIsBlacklistedForMalware) {