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

Unified Diff: chrome/browser/policy/configuration_policy_provider.h

Issue 10545033: Removed the PolicyDefinitionList from the ConfigurationPolicyProvider interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/configuration_policy_provider.h
diff --git a/chrome/browser/policy/configuration_policy_provider.h b/chrome/browser/policy/configuration_policy_provider.h
index d7b0e02e81e07aa1c34004a4a644226d95734465..2def6402d8ed212f84f22f524fdf6d7a17ac8a03 100644
--- a/chrome/browser/policy/configuration_policy_provider.h
+++ b/chrome/browser/policy/configuration_policy_provider.h
@@ -13,8 +13,6 @@
namespace policy {
-struct PolicyDefinitionList;
-
// A mostly-abstract super class for platform-specific policy providers.
// Platform-specific policy providers (Windows Group Policy, gconf,
// etc.) should implement a subclass of this class.
@@ -27,7 +25,7 @@ class ConfigurationPolicyProvider {
virtual void OnProviderGoingAway(ConfigurationPolicyProvider* provider);
};
- explicit ConfigurationPolicyProvider(const PolicyDefinitionList* policy_list);
+ ConfigurationPolicyProvider();
virtual ~ConfigurationPolicyProvider();
@@ -52,10 +50,6 @@ class ConfigurationPolicyProvider {
// The observers are notified after the policies are updated.
void UpdatePolicy(scoped_ptr<PolicyBundle> bundle);
- const PolicyDefinitionList* policy_definition_list() const {
- return policy_definition_list_;
- }
-
private:
friend class ConfigurationPolicyObserverRegistrar;
@@ -65,9 +59,6 @@ class ConfigurationPolicyProvider {
// The policies currently configured at this provider.
PolicyBundle policy_bundle_;
- // Contains the default mapping from policy values to the actual names.
- const PolicyDefinitionList* policy_definition_list_;
-
ObserverList<Observer, true> observer_list_;
DISALLOW_COPY_AND_ASSIGN(ConfigurationPolicyProvider);
« no previous file with comments | « chrome/browser/policy/config_dir_policy_provider_unittest.cc ('k') | chrome/browser/policy/configuration_policy_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698