Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1285)

Unified Diff: chrome/browser/extensions/management_policy.cc

Issue 82773002: Move some dependencies of ExtensionService down to extensions. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Presubmit check fixed Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/management_policy.cc
diff --git a/chrome/browser/extensions/management_policy.cc b/chrome/browser/extensions/management_policy.cc
deleted file mode 100644
index 2d169cb181c1aa6d5e61b228a6b75a22cde83514..0000000000000000000000000000000000000000
--- a/chrome/browser/extensions/management_policy.cc
+++ /dev/null
@@ -1,116 +0,0 @@
-// 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 {
-
-void GetExtensionNameAndId(const Extension* extension,
- std::string* name,
- std::string* id) {
- // The extension may be NULL in testing.
- *id = extension ? extension->id() : "[test]";
- *name = extension ? extension->name() : "test";
-}
-
-} // namespace
-
-ManagementPolicy::ManagementPolicy() {
-}
-
-ManagementPolicy::~ManagementPolicy() {
-}
-
-bool ManagementPolicy::Provider::UserMayLoad(const Extension* extension,
- string16* error) const {
- return true;
-}
-
-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;
-
- return false;
-}
-
-void ManagementPolicy::UnregisterAllProviders() {
- providers_.clear();
-}
-
-int ManagementPolicy::GetNumProviders() const {
- return providers_.size();
-}
-
-bool ManagementPolicy::ApplyToProviderList(ProviderFunction function,
- const char* debug_operation_name,
- bool normal_result,
- const Extension* extension,
- string16* error) const {
- for (ProviderList::const_iterator it = providers_.begin();
- it != providers_.end(); ++it) {
- const Provider* provider = *it;
- bool result = (provider->*function)(extension, error);
- if (result != normal_result) {
- std::string id;
- std::string name;
- GetExtensionNameAndId(extension, &name, &id);
- DVLOG(1) << debug_operation_name << " of extension " << name
- << " (" << id << ")"
- << " prohibited by " << provider->GetDebugPolicyProviderName();
- return !normal_result;
- }
- }
- return normal_result;
-}
-
-} // namespace extensions
« no previous file with comments | « chrome/browser/extensions/management_policy.h ('k') | chrome/browser/extensions/management_policy_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698