JSON Schema Compiler now supports conversion from Choice to base::Value.

tools/json_schema_compiler/h_generator.py

Index: tools/json_schema_compiler/h_generator.py
diff --git a/tools/json_schema_compiler/h_generator.py b/tools/json_schema_compiler/h_generator.py
index 5f59ffbe04603f83e730b78cf7bbec8c94698bb7..85ad37102f803da1fade85e7886204601fdab548 100644
--- a/tools/json_schema_compiler/h_generator.py
+++ b/tools/json_schema_compiler/h_generator.py
@@ -253,7 +253,8 @@ class HGenerator(object):
 
     if function.params:
       (c.Sblock('struct Params {')
-          .Concat(self._GeneratePropertyStructures(function.params))
+          .Concat(self._GeneratePropertyStructures(function.params,
+                                                   from_function=True))
           .Concat(self._GenerateFields(function.params))
           .Append('~Params();')
           .Append()
@@ -269,9 +270,10 @@ class HGenerator(object):
 
     return c
 
-  def _GeneratePropertyStructures(self, props):
+  def _GeneratePropertyStructures(self, props, from_function=False):
     """Generate the structures required by a property such as OBJECT classes
     and enums.
+    If from_function is True if these properties belong to an API Function.

[not at google - send to devlin, 2012/07/18 01:21:35]

You'll want to use from_client not from_function here, a property of the model
(i.e. props). That's what determines whether ToValue is generated.

(from_client means that the data originates on the client; meaning that we'll
want to ToValue it).

[chebert, 2012/07/19 00:56:01]

Done.

     """
     c = Code()
     for prop in props:
@@ -287,6 +289,11 @@ class HGenerator(object):
             prop,
             [choice.type_.name for choice in prop.choices.values()]))
         c.Concat(self._GeneratePropertyStructures(prop.choices.values()))
+        # We only need GetChoiceValue() if there is a ToValue() method.
+        if prop.from_json and not from_function:

[not at google - send to devlin, 2012/07/18 01:21:35]

I think this should all be "prop.from_client"?

[chebert, 2012/07/19 00:56:01]

Done.

+          c.Append('scoped_ptr<base::Value> GetChoiceValue(%s) const;' %

[not at google - send to devlin, 2012/07/18 01:21:35]

Btw this should be generated under private; should be easy enough to add an
entirely different function that does that?

_GeneratePrivatePropertyStructures...

so should CreateEnumValue actually (possible cleanup)

[chebert, 2012/07/19 00:56:01]

Turns out the CreateEnumValue move to private is not quick and easy like the
GetChoiceValue, so I will defer it to a future CL, in keeping with the small-CL
mantra.

On 2012/07/18 01:21:35, kalman wrote:

[not at google - send to devlin, 2012/07/19 01:48:22]

sure thing

+                   cpp_util.GetParameterDeclaration(prop,
+                       self._cpp_type_generator.GetChoicesEnumType(prop)))

[not at google - send to devlin, 2012/07/18 01:21:35]

There is some unfortunate whitespace thing going on in the generated code,
namely:

enum IntegersType {
        INTEGERS_NONE,
        INTEGERS_INTEGER,
        INTEGERS_ARRAY,
      };


      scoped_ptr<base::Value> GetChoiceValue(IntegersType integers) const;
      enum StringsType {
        STRINGS_ARRAY,
        STRINGS_STRING,
      };


      scoped_ptr<base::Value> GetChoiceValue(StringsType strings) const;
      IntegersType integers_type;

(check out previewserver.py)

The private thing might fix it.

[chebert, 2012/07/19 00:56:01]

Done.

       elif prop.type_ == PropertyType.ENUM:
         enum_name = self._cpp_type_generator.GetType(prop)
         c.Concat(self._GenerateEnumDeclaration(
@@ -308,7 +315,7 @@ class HGenerator(object):
     """
     c = Code()
     params = function.params
-    c.Concat(self._GeneratePropertyStructures(params))
+    c.Concat(self._GeneratePropertyStructures(params, from_function=True))
 
     param_lists = self._cpp_type_generator.GetAllPossibleParameterLists(params)
     for param_list in param_lists: