Support arguments in constructors.

client/dom/scripts/systemnative.py

 #!/usr/bin/python
 # Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
 """This module provides shared functionality for the systems to generate
 native binding from the IDL database."""
 
 import emitter
 import os
@@ skipping 237 matching lines @@
     self._interface_type_info = GetIDLTypeInfoByName(self._interface.id)
     self._members_emitter = emitter.Emitter()
     self._cpp_declarations_emitter = emitter.Emitter()
     self._cpp_impl_includes = {}
     self._cpp_definitions_emitter = emitter.Emitter()
     self._cpp_resolver_emitter = emitter.Emitter()
 
     self._GenerateConstructors()
 
   def _GenerateConstructors(self):
-    # WebKit IDLs may define constructors with arguments.  Currently this form is not supported
-    # (see b/1721).  There is custom implementation for some of them, the rest are just ignored
-    # for now.
-    SUPPORTED_CONSTRUCTORS_WITH_ARGS = [ 'WebKitCSSMatrix' ]
-    UNSUPPORTED_CONSTRUCTORS_WITH_ARGS = [
-        'EventSource',
-        'MediaStream',
-        'PeerConnection',
-        'ShadowRoot',
-        'SharedWorker',
-        'TextTrackCue',
-        'Worker' ]
-    if not self._IsConstructable() or self._interface.id in UNSUPPORTED_CONSTRUCTORS_WITH_ARGS:
+    if not self._IsConstructable():
       return
 
     # TODO(antonm): currently we don't have information about number of arguments expected by
     # the constructor, so name only dispatch.
     self._cpp_resolver_emitter.Emit(
         '    if (name == "$(INTERFACE_NAME)_constructor_Callback")\n'
         '        return Dart$(INTERFACE_NAME)Internal::constructorCallback;\n',
         INTERFACE_NAME=self._interface.id)
 
 
-    if self._interface.id in SUPPORTED_CONSTRUCTORS_WITH_ARGS or 'Constructor' not in self._interface.ext_attrs:
+    constructor_info = AnalyzeConstructor(self._interface)
+    if constructor_info is None:
       # We have a custom implementation for it.
       self._cpp_declarations_emitter.Emit(
           '\n'
           'void constructorCallback(Dart_NativeArguments);\n')
       return
 
     raises_dom_exceptions = 'ConstructorRaisesException' in self._interface.ext_attrs
-    raises_dart_exceptions = raises_dom_exceptions
+    raises_dart_exceptions = raises_dom_exceptions or len(constructor_info.idl_args) > 0
     type_info = GetIDLTypeInfo(self._interface)
     arguments = []
-    parameter_definitions = ''
+    parameter_definitions_emitter = emitter.Emitter()
     if 'CallWith' in self._interface.ext_attrs:
       call_with = self._interface.ext_attrs['CallWith']
       if call_with == 'ScriptExecutionContext':
         raises_dart_exceptions = True
-        parameter_definitions = (
+        parameter_definitions_emitter.Emit(
             '        ScriptExecutionContext* context = DartUtilities::scriptExecutionContext();\n'
             '        if (!context) {\n'
             '            exception = Dart_NewString("Failed to create an object");\n'
             '            goto fail;\n'
             '        }\n')
-        arguments = ['context']
+        arguments.append('context')
       else:
         raise Exception('Unsupported CallWith=%s attribute' % call_with)
 
+    # Process constructor arguments.
+    for (i, arg) in enumerate(constructor_info.idl_args):
+      self._GenerateParameterAdapter(parameter_definitions_emitter, arg, i - 1)
+      arguments.append(arg.id)
+
     self._GenerateNativeCallback(
         callback_name='constructorCallback',
         idl_node=self._interface,
-        parameter_definitions=parameter_definitions,
+        parameter_definitions=parameter_definitions_emitter.Fragments(),
         needs_receiver=False, function_name='%s::create' % type_info.native_type(),
         arguments=arguments,
         idl_return_type=self._interface,
         raises_dart_exceptions=raises_dart_exceptions,
         raises_dom_exceptions=raises_dom_exceptions)
 
 
   def _ImplClassName(self, interface_name):
     return interface_name + 'Implementation'
 
   def _IsConstructable(self):
     # FIXME: support ConstructorTemplate.
+    # FIXME: support NamedConstructor.
     return set(['CustomConstructor', 'V8CustomConstructor', 'Constructor']) & set(self._interface.ext_attrs)
 
   def FinishInterface(self):
     base = self._BaseClassName(self._interface)
     self._dart_impl_emitter.Emit(
         self._templates.Load('dart_implementation.darttemplate'),
         CLASS=self._class_name, BASE=base, INTERFACE=self._interface.id,
         MEMBERS=self._members_emitter.Fragments())
 
     self._GenerateCppHeader()
@@ skipping 441 matching lines @@
     namespace = 'HTMLNames'
     svg_exceptions = ['class', 'id', 'onabort', 'onclick', 'onerror', 'onload',
                       'onmousedown', 'onmousemove', 'onmouseout', 'onmouseover',
                       'onmouseup', 'onresize', 'onscroll', 'onunload']
     if self._interface.id.startswith('SVG') and not attr.id in svg_exceptions:
       namespace = 'SVGNames'
     self._cpp_impl_includes[namespace] = 1
 
     attribute_name = attr.ext_attrs['Reflect'] or attr.id.lower()
     return 'WebCore::%s::%sAttr' % (namespace, attribute_name)