Support scoped types in PPAPI IDL.

tools/json_schema_compiler/cc_generator.py

 # 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.
 
 from code import Code
 from model import PropertyType
 import cpp_util
 import schema_util
 import util_cc_helper
 
@@ skipping 668 matching lines @@
       )
       if is_ptr and self._generate_error_messages:
         c.Append('%(dst_var)s.reset();')
       else:
         c.Append('return %(failure_value)s;')
       c.Eblock('}')
       c.Sblock('else {')
       item_type = self._type_helper.FollowRef(underlying_type.item_type)
       if item_type.property_type == PropertyType.ENUM:
         c.Concat(self._GenerateListValueToEnumArrayConversion(
-                     item_type,
+                     underlying_type.item_type,  # Pass ref.

[not at google - send to devlin, 2014/03/28 18:42:25]

what does "Pass ref" mean?

[mtomasz, 2014/03/31 01:39:20]

PropertyType of the passed type is equal to REF. Removed, since confusing. Done.

                      'list',
                      dst_var,
                      failure_value,
                      is_ptr=is_ptr))
       else:
         (c.Sblock('if (!%s) {' % self._util_cc_helper.PopulateArrayFromList(
               underlying_type,
               'list',
               dst_var,
               is_ptr)))
@@ skipping 72 matching lines @@
     c = Code()
     accessor = '.'
     if is_ptr:
       accessor = '->'
       cpp_type = self._type_helper.GetCppType(item_type, is_in_container=True)
       c.Append('%s.reset(new std::vector<%s>);' %
                    (dst_var, cpp_util.PadForGenerics(cpp_type)))
     (c.Sblock('for (base::ListValue::const_iterator it = %s->begin(); '
                    'it != %s->end(); ++it) {' % (src_var, src_var))
       .Append('%s tmp;' % self._type_helper.GetCppType(item_type))
-      .Concat(self._GenerateStringToEnumConversion(item_type,
+      .Concat(self._GenerateStringToEnumConversion(item_type,  # Pass ref.
                                                    '(*it)',
                                                    'tmp',
                                                    failure_value))
       .Append('%s%spush_back(tmp);' % (dst_var, accessor))
       .Eblock('}')
     )
     return c
 
   def _GenerateStringToEnumConversion(self,
                                       type_,
                                       src_var,
                                       dst_var,
                                       failure_value):
     """Returns Code that converts a string type in |src_var| to an enum with
     type |type_| in |dst_var|. In the generated code, if |src_var| is not
     a valid enum name then the function will return |failure_value|.
     """
     c = Code()
     enum_as_string = '%s_as_string' % type_.unix_name
+    # The passed type may be either of REF type, or directly ENUM. Note, that
+    # the second case is used only by JSON and it is always in the default
+    # namespace.

[not at google - send to devlin, 2014/03/28 18:42:25]

It surprises me that enums need special treatment like this. what's different
about enums vs normal $ref aliases?

[mtomasz, 2014/03/29 04:22:53]

For enums we need to generate C++ code like this:
hello = namespace::ParseHello().

So we need to inject 'Parse' after the namespace, and before the type name.

[mtomasz, 2014/03/31 01:39:20]

I simplified the code. That's correct, that we don't need separate logic for
ENUM and REF. Done.

+    cpp_type_fullname = self._type_helper.GetCppType(type_)
+    cpp_type_namespace = None
+    cpp_type_name = None
+    # Eg. for other_namespace::EnumType:
+    # cpp_type_fullname = other_namespace::EnumType
+    # cpp_type_namespace = other_namespace
+    # cpp_type_name = EnumType
+    if type_.property_type == PropertyType.REF:
+      enum_type = self._type_helper.FollowRef(type_)
+      cpp_type_namespace = enum_type.namespace.unix_name
+      cpp_type_name = self._type_helper.GetCppType(enum_type)
+    else:
+      cpp_type_name = cpp_type_fullname
     (c.Append('std::string %s;' % enum_as_string)
       .Sblock('if (!%s->GetAsString(&%s)) {' % (src_var, enum_as_string))
       .Concat(self._GenerateError(
         '"\'%%(key)s\': expected string, got " + ' +
         self._util_cc_helper.GetValueTypeString('%%(src_var)s', True)))
       .Append('return %s;' % failure_value)
       .Eblock('}')
-      .Append('%s = Parse%s(%s);' % (dst_var,
-                                     self._type_helper.GetCppType(type_),
-                                     enum_as_string))
-      .Sblock('if (%s == %s) {' % (dst_var,
-                                 self._type_helper.GetEnumNoneValue(type_)))
+      .Append('%s = %sParse%s(%s);' % (
+        dst_var,
+        cpp_type_namespace + '::' if cpp_type_namespace else '',
+        cpp_type_name,
+        enum_as_string))
+      .Sblock('if (%s == %s%s) {' % (
+        dst_var,
+        cpp_type_namespace + '::' if cpp_type_namespace else '',
+        self._type_helper.GetEnumNoneValue(type_)))
       .Concat(self._GenerateError(
         '\"\'%%(key)s\': expected \\"' +
         '\\" or \\"'.join(
             enum_value.name
             for enum_value in self._type_helper.FollowRef(type_).enum_values) +
         '\\", got \\"" + %s + "\\""' % enum_as_string))
       .Append('return %s;' % failure_value)
       .Eblock('}')
       .Substitute({'src_var': src_var, 'key': type_.name})
     )
@@ skipping 145 matching lines @@
     if self._generate_error_messages:
       params = list(params) + ['base::string16* error']
     return ', '.join(str(p) for p in params)
 
   def _GenerateArgs(self, args):
     """Builds the argument list for a function, given an array of arguments.
     """
     if self._generate_error_messages:
       args = list(args) + ['error']
     return ', '.join(str(a) for a in args)