Chromium Code Reviews (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out

Unified Diff: components/policy/core/browser/configuration_policy_handler.h

Issue 197013007: Set drive as the default download folder (Closed) Base URL:
Patch Set: Fixes test failure and enables a new test Created 6 years, 9 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: components/policy/core/browser/configuration_policy_handler.h
diff --git a/components/policy/core/browser/configuration_policy_handler.h b/components/policy/core/browser/configuration_policy_handler.h
index b564d0e3cb270b59b8398ba7547ab82074937601..814f65b8f542273ccdd7a975b9afa858f833850a 100644
--- a/components/policy/core/browser/configuration_policy_handler.h
+++ b/components/policy/core/browser/configuration_policy_handler.h
@@ -21,6 +21,7 @@ class PrefValueMap;
namespace policy {
class PolicyErrorMap;
+struct PolicyHandlerParameters;
class PolicyMap;
// Maps a policy type to a preference path, and to the expected value type.
@@ -48,8 +49,17 @@ class POLICY_EXPORT ConfigurationPolicyHandler {
// Processes the policies handled by this ConfigurationPolicyHandler and sets
// the appropriate preferences in |prefs|.
+ virtual void ApplyPolicySettingsWithParameters(
+ const PolicyMap& policies,
+ const PolicyHandlerParameters& parameters,
+ PrefValueMap* prefs);
+ // This is a convenience version of ApplyPolicySettingsWithParameters()
+ // that leaves out the |parameters|. Anyone extending
+ // ConfigurationPolicyHandler should implement either ApplyPolicySettings or
+ // ApplyPolicySettingsWithParameters.
virtual void ApplyPolicySettings(const PolicyMap& policies,
- PrefValueMap* prefs) = 0;
+ PrefValueMap* prefs);
// Modifies the values of some of the policies in |policies| so that they
// are more suitable to display to the user. This can be used to remove
« no previous file with comments | « chrome/test/data/policy/policy_test_cases.json ('k') | components/policy/core/browser/ » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698