Added a PolicyManifestHandler.

chrome/browser/policy/policy_manifest_handler.h

+// 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_MANIFEST_HANDLER_H_
+#define CHROME_BROWSER_POLICY_POLICY_MANIFEST_HANDLER_H_
+
+#include "chrome/common/extensions/manifest_handler.h"
+
+namespace policy {
+
+class PolicyManifestHandler : public extensions::ManifestHandler {

[Mattias Nissler (ping if slow), 2013/05/15 10:37:39]

So chrome/browser/policy depends on chrome/browser/extensions again? And vice
versa?

Note that almost all ManifestHandlers are defined within
chrome/common/extensions.

[not at google - send to devlin, 2013/05/16 16:34:15]

Looks like this only depends on chrome/common/extensions to me.
The model Joao has is correct, the browser components project for extensions is
pulling the ManifestHandler implementations into where they are used. Eventually
the core of the manifest handling code will be moved in src/extensions/common.

[Mattias Nissler (ping if slow), 2013/05/16 18:27:54]

Fair, I was merely talking about dependencies between the policy and extensions
subsystems.
I'm totally on board with putting the ManifestHandlers next to the code that's
using the data the handler generates. By this argument, the handler belongs into
chrome/browser/extensions/api/storage, because that's where the schema
information is being used.

Side note: It should probably be renamed to StorageSchemaManifestHandler in any
case.

[Yoyo Zhou, 2013/05/16 19:14:16]

Yes, it's better to have policy extension API stuff in its own directory (either
under policy or extensions/api) because DEPS operate at the directory level.

[Joao da Silva, 2013/05/19 13:21:35]

Moved to
chrome/browser/extensions/api/storage/storage_schema_manifest_handler.cc.

+ public:
+  PolicyManifestHandler();
+  virtual ~PolicyManifestHandler();
+
+  // ManifestHandler implementation:
+  virtual bool Parse(extensions::Extension* extension,
+                     string16* error) OVERRIDE;
+  virtual bool Validate(
+      const extensions::Extension* extension,
+      std::string* error,
+      std::vector<extensions::InstallWarning>* warnings) const OVERRIDE;
+  virtual const std::vector<std::string> Keys() const OVERRIDE;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(PolicyManifestHandler);

[Mattias Nissler (ping if slow), 2013/05/15 10:37:39]

#include "base/basictypes.h"

[Joao da Silva, 2013/05/19 13:21:35]

Done.

+};
+
+}  // namespace policy
+
+#endif  // CHROME_BROWSER_POLICY_POLICY_MANIFEST_HANDLER_H_