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

Unified Diff: remoting/host/policy_watcher.h

Issue 2682473003: Add support for multiple allowed domains (Closed)
Patch Set: Rework to follow a deprecation approach Created 3 years, 8 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: remoting/host/policy_watcher.h
diff --git a/remoting/host/policy_watcher.h b/remoting/host/policy_watcher.h
index 5523b8478969d01b7c5d80c52896d0b5c0459d3f..44f38638de48f79f85f23d60b513c23587c0d000 100644
--- a/remoting/host/policy_watcher.h
+++ b/remoting/host/policy_watcher.h
@@ -92,12 +92,18 @@ class PolicyWatcher : public policy::PolicyService::Observer,
// Gets Chromoting schema stored inside |owned_schema_registry_|.
const policy::Schema* GetPolicySchema() const;
- // Simplifying wrapper around Schema::Normalize.
+ // Normalizes policies using Schema::Normalize and converts deprecated
+ // policies.
+ //
// - Returns false if |dict| is invalid (i.e. contains mistyped policy
// values).
// - Returns true if |dict| was valid or got normalized.
bool NormalizePolicies(base::DictionaryValue* dict);
+ // Converts each deprecated policy to its replacement if and only if the
+ // replacement policy is not set, and removes deprecated policied from dict.
+ void HandleDeprecatedPolicies(base::DictionaryValue* dict);
+
// Stores |new_policies| into |old_policies_|. Returns dictionary with items
// from |new_policies| that are different from the old |old_policies_|.
std::unique_ptr<base::DictionaryValue> StoreNewAndReturnChangedPolicies(

Powered by Google App Engine
This is Rietveld 408576698