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

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

Issue 15061007: Added a PolicyDomainDescriptor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: addressed comments, rebase on new PolicySchema Created 7 years, 7 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/policy_domain_descriptor.h
diff --git a/chrome/browser/policy/policy_domain_descriptor.h b/chrome/browser/policy/policy_domain_descriptor.h
new file mode 100644
index 0000000000000000000000000000000000000000..022dfe2678ab57989f311994876acc732eb0f105
--- /dev/null
+++ b/chrome/browser/policy/policy_domain_descriptor.h
@@ -0,0 +1,53 @@
+// Copyright (c) 2013 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.
+
+#ifndef CHROME_BROWSER_POLICY_POLICY_DOMAIN_DESCRIPTOR_H_
+#define CHROME_BROWSER_POLICY_POLICY_DOMAIN_DESCRIPTOR_H_
+
+#include <map>
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/policy/policy_service.h"
+
+namespace policy {
+
+class PolicyBundle;
+class PolicySchema;
+
+// For each policy domain, this class keeps the complete list of valid
+// components for that domain, and the PolicySchema for each component.
+class PolicyDomainDescriptor : public base::RefCounted<PolicyDomainDescriptor> {
+ public:
+ typedef std::map<std::string, const PolicySchema*> SchemaMap;
+
+ explicit PolicyDomainDescriptor(PolicyDomain domain);
+
+ PolicyDomain domain() const { return domain_; }
+ const SchemaMap& components() const { return schema_map_; }
+
+ // Sets the |schema| for |component_id|, overridding any previously set
+ // schema for that component.
+ void SetComponent(const std::string& component_id,
bartfab (slow) 2013/05/21 12:10:56 Nit: Maybe better |SetComponentSchema| or |AddComp
Joao da Silva 2013/05/21 17:50:14 Went for RegisterComponent(), I think that conveys
+ scoped_ptr<PolicySchema> schema);
+
+ // Removes all the policies in |bundle| that don't match this descriptor.
+ // Policies of domains other than |domain_| are ignored.
+ void FilterBundle(PolicyBundle* bundle) const;
+
+ private:
+ friend class base::RefCounted<PolicyDomainDescriptor>;
+ ~PolicyDomainDescriptor();
+
+ PolicyDomain domain_;
+ SchemaMap schema_map_;
+
+ DISALLOW_COPY_AND_ASSIGN(PolicyDomainDescriptor);
+};
+
+} // namespace policy
+
+#endif // CHROME_BROWSER_POLICY_POLICY_DOMAIN_DESCRIPTOR_H_

Powered by Google App Engine
This is Rietveld 408576698