Code generation for extensions api

chrome/browser/extensions/extension_permissions_api_helpers.h

 // Copyright (c) 2012 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_EXTENSIONS_EXTENSION_PERMISSIONS_API_HELPERS_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_PERMISSIONS_API_HELPERS_H_
 #pragma once
 
 #include <string>
 
-#include "base/memory/ref_counted.h"
-
 namespace base {
 class DictionaryValue;
 }
 class ExtensionPermissionSet;
 
 namespace extensions {
+namespace permissions_api {

[Aaron Boodman, 2012/01/19 23:31:13]

Maybe you should reserve api::<name> for generated code? That way names won't
conflict. This could just be part of extensions namespace.

[calamity, 2012/01/23 05:14:45]

Done.

 
-namespace permissions_api {
+struct Permissions;
 
 // Converts the permission |set| to a dictionary value.
 base::DictionaryValue* PackPermissionsToValue(
     const ExtensionPermissionSet* set);
 
-// Converts the |value| to a permission set.
-bool UnpackPermissionsFromValue(base::DictionaryValue* value,
-                                scoped_refptr<ExtensionPermissionSet>* ptr,
-                                bool* bad_message,
-                                std::string* error);
+// Converts the |permissions| to a new permission set. Returns NULL if the

[Yoyo Zhou, 2012/01/19 02:19:40]

What do you mean by "new" permission set?

[calamity, 2012/01/20 01:10:25]

In the sense that the function creates the ExtensionPermissionSet.

+// permissions cannot be converted to a permission set, in which case |error|
+// will be set.
+ExtensionPermissionSet* UnpackPermissionSet(
+    const Permissions& permissions, std::string* error);
 
 }  // namespace permissions_api
-
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_PERMISSIONS_API_HELPERS_H__