Fixes for Chrome 34 merge

tools/dom/scripts/generator.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 systems to generate
 Dart APIs from the IDL database."""
 
 import copy
 import json
@@ skipping 52 matching lines @@
     'generator._methods_with_named_formals', [
   'DirectoryEntry.getDirectory',
   'DirectoryEntry.getFile',
   'Entry.copyTo',
   'Entry.moveTo',
   'HTMLInputElement.setRangeText',
   'HTMLTextAreaElement.setRangeText',
   'XMLHttpRequest.open',
   ])
 
+def ReturnValueConversionHack(idl_type, value, interface_name):
+  if idl_type == 'SVGMatrix':
+    return '%sTearOff::create(%s)' % (idl_type, value)
+  elif ((idl_type == 'SVGAngle' and interface_name != 'SVGAnimatedAngle')
+      or (idl_type == 'SVGTransform' and interface_name == 'SVGSVGElement')):
+    # Somewhere in the IDL it probably specifies whether we need to call
+    # create or not.
+    return 'SVGPropertyTearOff<%s>::create(%s)' % (idl_type, value)
+
+  return value
+
 #
 # Renames for attributes that have names that are not legal Dart names.
 #
 _dart_attribute_renames = monitored.Dict('generator._dart_attribute_renames', {
     'default': 'defaultValue',
 })
 
 #
 # Interface version of the DOM needs to delegate typed array constructors to a
 # factory provider.
@@ skipping 572 matching lines @@
 
   def native_type(self):
     return self._data.native_type or self._idl_type
 
   def bindings_class(self):
     return 'Dart%s' % self.idl_type()
 
   def vector_to_dart_template_parameter(self):
     return self.native_type()
 
-  def to_native_info(self, idl_node, interface_name):
+  def to_native_info(self, idl_node, interface_name, callback_name):
     cls = self.bindings_class()
 
     if 'Callback' in idl_node.ext_attrs:
       return '%s.release()', 'OwnPtr<%s>' % self.native_type(), cls, 'create'
 
-    if self.custom_to_native():
+    # This is a hack to handle property references correctly.
+    if (self.native_type() in ['SVGPropertyTearOff<SVGAngle>',
+          'SVGPropertyTearOff<SVGAngle>*', 'SVGMatrixTearOff']
+        and (callback_name != 'createSVGTransformFromMatrixCallback'
+          or interface_name != 'SVGTransformList')):
+      argument_expression_template = '%s->propertyReference()'
+      type = '%s*' % self.native_type()
+    elif self.custom_to_native():
       type = 'RefPtr<%s>' % self.native_type()
       argument_expression_template = '%s.get()'
     else:
       type = '%s*' % self.native_type()
-      if isinstance(self, SVGTearOffIDLTypeInfo) and not interface_name.endswith('List'):
-        argument_expression_template = '%s->propertyReference()'
-      else:
-        argument_expression_template = '%s'
+      argument_expression_template = '%s'
     return argument_expression_template, type, cls, 'toNative'
 
   def pass_native_by_ref(self): return False
 
   def custom_to_native(self):
     return self._data.custom_to_native
 
   def parameter_type(self):
     return '%s*' % self.native_type()
 
@@ skipping 32 matching lines @@
     includes = [self._idl_type] + (self._data.conversion_includes or [])
     return ['"Dart%s.h"' % include for include in includes]
 
   def to_dart_conversion(self, value, interface_name=None, attributes=None):
     return 'Dart%s::toDart(%s)' % (self._idl_type, value)
 
   def return_to_dart_conversion(self, value, auto_dart_scope_setup,
                                 interface_name=None, attributes=None):
     auto_dart_scope='true' if auto_dart_scope_setup else 'false'
     return 'Dart%s::returnToDart(args, %s, %s)' % (self._idl_type,
-                                                   value,
+                                                   ReturnValueConversionHack(self._idl_type, value, interface_name),
                                                    auto_dart_scope)
 
   def custom_to_dart(self):
     return self._data.custom_to_dart
 
 
 class InterfaceIDLTypeInfo(IDLTypeInfo):
   def __init__(self, idl_type, data, dart_interface_name, type_registry):
     super(InterfaceIDLTypeInfo, self).__init__(idl_type, data)
     self._dart_interface_name = dart_interface_name
@@ skipping 88 matching lines @@
 
   def interface_name(self):
     return self.dart_type()
 
   def implementation_name(self):
     return self.dart_type()
 
   def vector_to_dart_template_parameter(self):
     raise Exception('sequences of sequences are not supported yet')
 
-  def to_native_info(self, idl_node, interface_name):
+  def to_native_info(self, idl_node, interface_name, callback_name):
     item_native_type = self._item_info.vector_to_dart_template_parameter()
     if isinstance(self._item_info, PrimitiveIDLTypeInfo):
       return '%s', 'Vector<%s>' % item_native_type, 'DartUtilities', 'toNativeVector<%s>' % item_native_type
     return '%s', 'Vector< RefPtr<%s> >' % item_native_type, 'DartUtilities', 'toNativeVector< RefPtr<%s> >' % item_native_type
 
   def parameter_type(self):
     native_type = self.native_type()
     if array_type(native_type):
       return 'const Vector<RefPtr<%s> > &' % array_type(native_type)
 
@@ skipping 14 matching lines @@
         attributes)
 
   def conversion_includes(self):
     return self._item_info.conversion_includes()
 
 
 class DOMStringArrayTypeInfo(SequenceIDLTypeInfo):
   def __init__(self, data, item_info):
     super(DOMStringArrayTypeInfo, self).__init__('DOMString[]', data, item_info)
 
-  def to_native_info(self, idl_node, interface_name):
+  def to_native_info(self, idl_node, interface_name, callback_name):
     return '%s', 'RefPtr<DOMStringList>', 'DartDOMStringList', 'toNative'
 
   def pass_native_by_ref(self): return False
 
   def implementation_name(self):
     return ""
 
 
 class PrimitiveIDLTypeInfo(IDLTypeInfo):
   def __init__(self, idl_type, data):
     super(PrimitiveIDLTypeInfo, self).__init__(idl_type, data)
 
   def vector_to_dart_template_parameter(self):
     # Ugly hack. Usually IDLs floats are treated as C++ doubles, however
     # sequence<float> should map to Vector<float>
     if self.idl_type() == 'float': return 'float'
     return self.native_type()
 
-  def to_native_info(self, idl_node, interface_name):
+  def to_native_info(self, idl_node, interface_name, callback_name):
     type = self.native_type()
     if type == 'SerializedScriptValue':
       type = 'RefPtr<%s>' % type
     if type == 'String':
       type = 'DartStringAdapter'
     target_type = self._capitalized_native_type()
     if self.idl_type() == 'Date':
       target_type = 'Date'
     return '%s', type, 'DartUtilities', 'dartTo%s' % target_type
 
@@ skipping 43 matching lines @@
 
   def native_type(self):
     if self._data.native_type:
       return self._data.native_type
     tear_off_type = 'SVGPropertyTearOff'
     if self._idl_type.endswith('List'):
       tear_off_type = 'SVGListPropertyTearOff'
     return '%s<%s>' % (tear_off_type, self._idl_type)
 
   def receiver(self):
-    if self._idl_type.endswith('List'):
-      return 'receiver->'
-    return 'receiver->propertyReference().'
+    return 'receiver->'
 
   def to_conversion_cast(self, value, interface_name, attributes):
-    svg_primitive_types = ['SVGAngle', 'SVGLength', 'SVGMatrix',
+    svg_primitive_types = ['SVGLength', 'SVGMatrix',
         'SVGNumber', 'SVGPoint', 'SVGRect', 'SVGTransform']
-    conversion_cast = '%s::create(%s)'
-    if interface_name.startswith('SVGAnimated'):
-      conversion_cast = 'static_cast<%s*>(%s)'
-    elif self.idl_type() == 'SVGStringList':
-      conversion_cast = '%s::create(receiver, %s)'
-    elif interface_name.endswith('List'):
-      conversion_cast = 'static_cast<%s*>(%s.get())'
-    elif self.idl_type() in svg_primitive_types:
-      conversion_cast = '%s::create(%s)'
-    else:
-      conversion_cast = 'static_cast<%s*>(%s)'
+
+    # This is a hack. We either need to figure out the right way to derive this
+    # information from the IDL or remove this generator.
+    if self.idl_type() != 'SVGTransformList':
+      return value
+
+    conversion_cast = 'static_cast<%s*>(%s)'
     conversion_cast = conversion_cast % (self.native_type(), value)
     return '%s' % (conversion_cast)
 
   def to_dart_conversion(self, value, interface_name, attributes):
     return 'Dart%s::toDart(%s)' % (self._idl_type, self.to_conversion_cast(value, interface_name, attributes))
 
   def return_to_dart_conversion(self, value, auto_dart_scope_setup,
                                 interface_name, attr):
     auto_dart_scope='true' if auto_dart_scope_setup else 'false'
     return 'Dart%s::returnToDart(args, %s, %s)' % (self._idl_type,
                                                    self.to_conversion_cast(
-                                                       value,
+                                                       ReturnValueConversionHack(self._idl_type, value, interface_name),
                                                        interface_name,
                                                        attr),
                                                    auto_dart_scope)
 
   def argument_expression(self, name, interface_name):
-    return name if interface_name.endswith('List') else '%s->propertyReference()' % name
-
+    return name
 
 class TypedListIDLTypeInfo(InterfaceIDLTypeInfo):
   def __init__(self, idl_type, data, interface_name, type_registry):
     super(TypedListIDLTypeInfo, self).__init__(
         idl_type, data, interface_name, type_registry)
 
   def conversion_includes(self):
     return [ '"wtf/%s.h"' % self._idl_type ]
 
   def to_dart_conversion(self, value, interface_name, attributes):
     return 'DartUtilities::arrayBufferViewToDart(%s)' % value
 
   def return_to_dart_conversion(self, value, auto_dart_scope_setup,
                                 interface_name, attributes):
     return 'Dart_SetReturnValue(args, %s)' % self.to_dart_conversion(
         value,
         interface_name,
         attributes)
 
-  def to_native_info(self, idl_node, interface_name):
+  def to_native_info(self, idl_node, interface_name, callback_name):
     return '%s.get()', 'RefPtr<%s>' % self._idl_type, 'DartUtilities', 'dartTo%s' % self._idl_type
 
 
 class BasicTypedListIDLTypeInfo(InterfaceIDLTypeInfo):
   def __init__(self, idl_type, data, interface_name, type_registry):
     super(BasicTypedListIDLTypeInfo, self).__init__(
         idl_type, data, interface_name, type_registry)
 
   def conversion_includes(self):
     return []
 
   def to_dart_conversion(self, value, interface_name, attributes):
     function_name = 'DartUtilities::%sToDart' % self._idl_type
     function_name = function_name[0].lower() + function_name[1:]
     return '%s(%s)' % (function_name, value)
 
   def return_to_dart_conversion(self, value, auto_dart_scope_setup,
                                 interface_name, attributes):
     return 'Dart_SetReturnValue(args, %s)' % self.to_dart_conversion(
         value,
         interface_name,
         attributes)
 
-  def to_native_info(self, idl_node, interface_name):
+  def to_native_info(self, idl_node, interface_name, callback_name):
     return '%s.get()', 'RefPtr<%s>' % self._idl_type, 'DartUtilities', 'dartTo%s' % self._idl_type
 
 
 class TypeData(object):
   def __init__(self, clazz, dart_type=None, native_type=None,
                merged_interface=None, merged_into=None,
                custom_to_dart=False, custom_to_native=False,
                conversion_includes=None,
                webcore_getter_name='getAttribute',
                webcore_setter_name='setAttribute',
@@ skipping 119 matching lines @@
     'Int16Array': TypedListTypeData('int'),
     'Int32Array': TypedListTypeData('int'),
     'Uint8Array': TypedListTypeData('int'),
     'Uint8ClampedArray': TypedListTypeData('int'),
     'Uint16Array': TypedListTypeData('int'),
     'Uint32Array': TypedListTypeData('int'),
 
     'ArrayBufferView': TypeData(clazz='BasicTypedList'),
     'ArrayBuffer': TypeData(clazz='BasicTypedList'),
 
-    'SVGAngle': TypeData(clazz='SVGTearOff'),
-    'SVGLength': TypeData(clazz='SVGTearOff'),
-    'SVGLengthList': TypeData(clazz='SVGTearOff', item_type='SVGLength'),
-    'SVGMatrix': TypeData(clazz='SVGTearOff'),
-    'SVGNumber': TypeData(clazz='SVGTearOff', native_type='SVGPropertyTearOff<SVGNumber>'),
-    'SVGNumberList': TypeData(clazz='SVGTearOff', item_type='SVGNumber'),
+    'SVGAngle': TypeData(clazz='SVGTearOff', native_type='SVGPropertyTearOff<SVGAngle>'),
+    'SVGLength': TypeData(clazz='SVGTearOff', native_type='SVGLengthTearOff'),
+    'SVGLengthList': TypeData(clazz='SVGTearOff', item_type='SVGLength', native_type='SVGLengthListTearOff'),
+    'SVGMatrix': TypeData(clazz='SVGTearOff', native_type='SVGMatrixTearOff'),
+    'SVGNumber': TypeData(clazz='SVGTearOff', native_type='SVGNumberTearOff'),
+    'SVGNumberList': TypeData(clazz='SVGTearOff', item_type='SVGNumber', native_type='SVGNumberListTearOff'),
     'SVGPathSegList': TypeData(clazz='SVGTearOff', item_type='SVGPathSeg',
         native_type='SVGPathSegListPropertyTearOff'),
-    'SVGPoint': TypeData(clazz='SVGTearOff', native_type='SVGPropertyTearOff<FloatPoint>'),
-    'SVGPointList': TypeData(clazz='SVGTearOff'),
-    'SVGPreserveAspectRatio': TypeData(clazz='SVGTearOff'),
-    'SVGRect': TypeData(clazz='SVGTearOff', native_type='SVGPropertyTearOff<SVGRect>'),
+    'SVGPoint': TypeData(clazz='SVGTearOff', native_type='SVGPointTearOff'),
+    'SVGPointList': TypeData(clazz='SVGTearOff', native_type='SVGPointListTearOff'),
+    'SVGPreserveAspectRatio': TypeData(clazz='SVGTearOff', native_type='SVGPreserveAspectRatioTearOff'),
+    'SVGRect': TypeData(clazz='SVGTearOff', native_type='SVGRectTearOff'),
     'SVGStringList': TypeData(clazz='SVGTearOff', item_type='DOMString',
-        native_type='SVGStaticListPropertyTearOff<SVGStringList>'),
-    'SVGTransform': TypeData(clazz='SVGTearOff'),
+        native_type='SVGStringListTearOff'),
+    'SVGTransform': TypeData(clazz='SVGTearOff', native_type="SVGPropertyTearOff<SVGTransform>"),
     'SVGTransformList': TypeData(clazz='SVGTearOff', item_type='SVGTransform',
         native_type='SVGTransformListPropertyTearOff'),
 })
 
 _svg_supplemental_includes = [
-    '"SVGAnimatedPropertyTearOff.h"',
-    '"SVGAnimatedListPropertyTearOff.h"',
-    '"SVGStaticListPropertyTearOff.h"',
-    '"SVGAnimatedListPropertyTearOff.h"',
-    '"SVGTransformListPropertyTearOff.h"',
-    '"SVGPathSegListPropertyTearOff.h"',
+    '"core/svg/properties/SVGPropertyTraits.h"',
 ]
 
 class TypeRegistry(object):
   def __init__(self, database, renamer=None):
     self._database = database
     self._renamer = renamer
     self._cache = {}
 
   def TypeInfo(self, type_name):
     if not type_name in self._cache:
@@ skipping 54 matching lines @@
     if type_data.clazz == 'BasicTypedList':
       if type_name == 'ArrayBuffer':
         dart_interface_name = 'ByteBuffer'
       else:
         dart_interface_name = self._renamer.RenameInterfaceId(type_name)
       return BasicTypedListIDLTypeInfo(
           type_name, type_data, dart_interface_name, self)
 
     class_name = '%sIDLTypeInfo' % type_data.clazz
     return globals()[class_name](type_name, type_data)