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

Side by Side Diff: chrome/browser/extensions/management_policy.cc

Issue 10542023: Disable modifying extensions when in managed mode. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Fixed Evan's nits Created 8 years, 6 months 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/extensions/management_policy.h" 5 #include "chrome/browser/extensions/management_policy.h"
6 6
7 namespace extensions { 7 namespace extensions {
8 ManagementPolicy::ManagementPolicy() { 8 ManagementPolicy::ManagementPolicy() {
9 } 9 }
10 10
(...skipping 23 matching lines...) Expand all
34 providers_.erase(provider); 34 providers_.erase(provider);
35 } 35 }
36 36
37 bool ManagementPolicy::UserMayLoad(const Extension* extension, 37 bool ManagementPolicy::UserMayLoad(const Extension* extension,
38 string16* error) const { 38 string16* error) const {
39 for (ProviderList::const_iterator it = providers_.begin(); 39 for (ProviderList::const_iterator it = providers_.begin();
40 it != providers_.end(); ++it) { 40 it != providers_.end(); ++it) {
41 if (!(*it)->UserMayLoad(extension, error)) { 41 if (!(*it)->UserMayLoad(extension, error)) {
42 // The extension may be NULL in testing. 42 // The extension may be NULL in testing.
43 std::string id = extension ? extension->id() : "[test]"; 43 std::string id = extension ? extension->id() : "[test]";
44 DLOG(WARNING) << "Installation of extension " << id 44 std::string name = extension ? extension->name() : "test";
45 << " prohibited by " << (*it)->GetPolicyProviderName(); 45 DLOG(WARNING) << "Installation of extension " << name
46 << "( " << id << ")"
47 << " prohibited by " << (*it)->GetDebugPolicyProviderName();
46 return false; 48 return false;
47 } 49 }
48 } 50 }
49 return true; 51 return true;
50 } 52 }
51 53
52 bool ManagementPolicy::UserMayModifySettings(const Extension* extension, 54 bool ManagementPolicy::UserMayModifySettings(const Extension* extension,
53 string16* error) const { 55 string16* error) const {
54 for (ProviderList::const_iterator it = providers_.begin(); 56 for (ProviderList::const_iterator it = providers_.begin();
55 it != providers_.end(); ++it) { 57 it != providers_.end(); ++it) {
56 if (!(*it)->UserMayModifySettings(extension, error)) { 58 if (!(*it)->UserMayModifySettings(extension, error)) {
57 // The extension may be NULL in testing. 59 // The extension may be NULL in testing.
58 std::string id = extension ? extension->id() : "[test]"; 60 std::string id = extension ? extension->id() : "[test]";
59 DLOG(WARNING) << "Modification of extension " << id 61 std::string name = extension ? extension->name() : "test";
60 << " prohibited by " << (*it)->GetPolicyProviderName(); 62 DLOG(WARNING) << "Modification of extension " << name
63 << "( " << id << ")"
64 << " prohibited by " << (*it)->GetDebugPolicyProviderName();
61 return false; 65 return false;
62 } 66 }
63 } 67 }
64 return true; 68 return true;
65 } 69 }
66 70
67 bool ManagementPolicy::MustRemainEnabled(const Extension* extension, 71 bool ManagementPolicy::MustRemainEnabled(const Extension* extension,
68 string16* error) const { 72 string16* error) const {
69 for (ProviderList::const_iterator it = providers_.begin(); 73 for (ProviderList::const_iterator it = providers_.begin();
70 it != providers_.end(); ++it) { 74 it != providers_.end(); ++it) {
75 if ((*it)->MustRemainEnabled(extension, error)) {
71 // The extension may be NULL in testing. 76 // The extension may be NULL in testing.
72 std::string id = extension ? extension->id() : "[test]"; 77 std::string id = extension ? extension->id() : "[test]";
73 if ((*it)->MustRemainEnabled(extension, error)) { 78 std::string name = extension ? extension->name() : "test";
74 DLOG(WARNING) << "Extension " << id 79 DLOG(WARNING) << "Extension " << name
80 << "( " << id << ")"
75 << " required to remain enabled by " 81 << " required to remain enabled by "
76 << (*it)->GetPolicyProviderName(); 82 << (*it)->GetDebugPolicyProviderName();
77 return true; 83 return true;
78 } 84 }
79 } 85 }
80 return false; 86 return false;
81 } 87 }
82 88
83 void ManagementPolicy::UnregisterAllProviders() { 89 void ManagementPolicy::UnregisterAllProviders() {
84 providers_.clear(); 90 providers_.clear();
85 } 91 }
86 92
87 int ManagementPolicy::GetNumProviders() const { 93 int ManagementPolicy::GetNumProviders() const {
88 return providers_.size(); 94 return providers_.size();
89 } 95 }
90 } // namespace 96 } // namespace
OLDNEW
« no previous file with comments | « chrome/browser/extensions/management_policy.h ('k') | chrome/browser/extensions/test_management_policy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698