Add management policy function allowing extensions to be disabled.

chrome/browser/extensions/management_policy.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/management_policy.h"
 
 namespace extensions {
 
 namespace {
 
@@ skipping 21 matching lines @@
 bool ManagementPolicy::Provider::UserMayModifySettings(
     const Extension* extension, string16* error) const {
   return true;
 }
 
 bool ManagementPolicy::Provider::MustRemainEnabled(const Extension* extension,
                                                    string16* error) const {
   return false;
 }
 
+bool ManagementPolicy::Provider::MustRemainDisabled(
+    const Extension* extension,
+    Extension::DisableReason* reason,
+    string16* error) const {
+  return false;
+}
+
 void ManagementPolicy::RegisterProvider(Provider* provider) {
   providers_.insert(provider);
 }
 
 void ManagementPolicy::UnregisterProvider(Provider* provider) {
   providers_.erase(provider);
 }
 
 bool ManagementPolicy::UserMayLoad(const Extension* extension,
                                    string16* error) const {
   return ApplyToProviderList(&Provider::UserMayLoad, "Installation",
                              true, extension, error);
 }
 
 bool ManagementPolicy::UserMayModifySettings(const Extension* extension,
                                              string16* error) const {
   return ApplyToProviderList(&Provider::UserMayModifySettings, "Modification",
                              true, extension, error);
 }
 
 bool ManagementPolicy::MustRemainEnabled(const Extension* extension,
                                          string16* error) const {
   return ApplyToProviderList(&Provider::MustRemainEnabled, "Disabling",
                              false, extension, error);
 }
 
+bool ManagementPolicy::MustRemainDisabled(const Extension* extension,
+                                          Extension::DisableReason* reason,
+                                          string16* error) const {
+  for (ProviderList::const_iterator it = providers_.begin();
+       it != providers_.end(); ++it)
+    if ((*it)->MustRemainDisabled(extension, reason, error))
+      return true;

[miket_OOO, 2013/11/04 23:18:10]

See any_of. http://www.cplusplus.com/reference/algorithm/ (not sure whether it
maps nicely to this case, but it seems like a candidate)

[asargent_no_longer_on_chrome, 2013/11/05 00:20:44]

I looked into it briefly, but it would be very awkward for this case because
any_of takes a UnaryPredicate function which takes only a single input (an
element of the collection). But in this case I need to call a method on each of
the objects kept in the collection, passing that method the 3 arguments passed
in to this method. I don't think our currying support in base/callback.h
supports binding a class method but leaving the object reference unbound while
binding further arguments to the method. 

+
+  return false;
+}
+
 void ManagementPolicy::UnregisterAllProviders() {
   providers_.clear();
 }
 
 int ManagementPolicy::GetNumProviders() const {
   return providers_.size();
 }
 
 bool ManagementPolicy::ApplyToProviderList(ProviderFunction function,
                                            const char* debug_operation_name,
@@ skipping 11 matching lines @@
       DVLOG(1) << debug_operation_name << " of extension " << name
                << " (" << id << ")"
                << " prohibited by " << provider->GetDebugPolicyProviderName();
       return !normal_result;
     }
   }
   return normal_result;
 }
 
 }  // namespace extensions