Support array types for function and callback arguments in IDL APIs.

tools/json_schema_compiler/idl_schema.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.
 
 import os.path
 import sys
 
 # This file is a peer to json_schema.py. Each of these files understands a
 # certain format describing APIs (either JSON or IDL), reads files written
 # in that format into memory, and emits them as a Python array of objects
@@ skipping 90 matching lines @@
   function parameter, converts into a Python dictionary that the JSON schema
   compiler expects to see.
   '''
   def __init__(self, typeref, parent, additional_properties={}):
     self.typeref = typeref
     self.parent = parent
     self.additional_properties = additional_properties
 
   def process(self, refs):
     properties = self.additional_properties
+    result = properties
 
     if self.parent.GetProperty('OPTIONAL', False):
       properties['optional'] = True
 
+    # The IDL parser denotes array types by adding a child 'Array' node onto
+    # the Param node in the Callspec.
+    for sibling in self.parent.GetChildren():
+      if sibling.cls == 'Array' and sibling.GetName() == self.parent.GetName():
+        properties['type'] = 'array'
+        properties['items'] = {}
+        properties = properties['items']
+        break
+
     if self.typeref == 'DOMString':
       properties['type'] = 'string'
     elif self.typeref == 'boolean':
       properties['type'] = 'boolean'
     elif self.typeref == 'long':
       properties['type'] = 'integer'
     elif self.typeref == 'any':
       properties['type'] = 'any'
     elif self.typeref == 'object':
       properties['type'] = 'object'
       if 'additionalProperties' not in properties:
         properties['additionalProperties'] = {}
       properties['additionalProperties']['type'] = 'any'
       instance_of = self.parent.GetProperty('instanceOf')
       if instance_of:
         properties['isInstanceOf'] = instance_of
     elif self.typeref is None:
       properties['type'] = 'function'
     else:
       try:
-        properties = refs[self.typeref]
+        result = refs[self.typeref]

[Matt Perry, 2012/04/12 18:56:53]

What is this code path used for? Doesn't this miss the array case? It seems like
for an array, we should do properties['items'] = refs[self.typeref]

       except KeyError, e:
         properties['$ref'] = self.typeref
-    return properties
+
+    return result
 
 class Namespace(object):
   '''
   Given an IDLNode representing an IDL namespace, converts into a Python
   dictionary that the JSON schema compiler expects to see.
   '''
 
   def __init__(self, namespace_node, nodoc=False):
     self.namespace = namespace_node
     self.nodoc = nodoc
@@ skipping 67 matching lines @@
   Python dictionary in a format that the JSON schema compiler expects to see.
   '''
 
   f = open(filename, 'r')
   contents = f.read()
   f.close()
 
   idl = idl_parser.IDLParser().ParseData(contents, filename)
   idl_schema = IDLSchema(idl)
   return idl_schema.process()