Generate storage API constants using json_schema_compiler.

TEST=unit_tests --gtest_filter=*JsonSchema*


Review URL: http://codereview.chromium.org/10116015

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132945 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 11 insertions, 8 deletions

diff --git a/chrome/renderer/resources/extensions/schema_generated_bindings.js b/chrome/renderer/resources/extensions/schema_generated_bindings.js
index 4da0dba..66c193a 100644
--- a/chrome/renderer/resources/extensions/schema_generated_bindings.js
+++ b/chrome/renderer/resources/extensions/schema_generated_bindings.js
@@ -522,9 +522,14 @@
 
           var value = propertyDef.value;
           if (value) {
-            if (propertyDef.type === 'integer') {
+            // Values may just have raw types as defined in the JSON, such
+            // as "WINDOW_ID_NONE": { "value": -1 }. We handle this here.
+            // TODO(kalman): enforce that things with a "value" property can't
+            // define their own types.
+            var type = propertyDef.type || typeof(value);
+            if (type === 'integer' || type === 'number') {
               value = parseInt(value);
-            } else if (propertyDef.type === 'boolean') {
+            } else if (type === 'boolean') {
               value = value === "true";
             } else if (propertyDef["$ref"]) {
               var constructor = customTypes[propertyDef["$ref"]];
@@ -539,15 +544,13 @@
               constructor.apply(value, args);
               // Recursively add properties.
               addProperties(value, propertyDef);
-            } else if (propertyDef.type === 'object') {
+            } else if (type === 'object') {
               // Recursively add properties.
               addProperties(value, propertyDef);
-            } else if (propertyDef.type !== 'string') {
-              throw "NOT IMPLEMENTED (extension_api.json error): Cannot " +
-                  "parse values for type \"" + propertyDef.type + "\"";
+            } else if (type !== 'string') {
+              throw new Error("NOT IMPLEMENTED (extension_api.json error): " +
+                  "Cannot parse values for type \"" + type + "\"");
             }
-          }
-          if (value) {
             m[propertyName] = value;
           }
         });