IDL compiler: [CallWith] for setters

Source/bindings/scripts/unstable/v8_attributes.py

 # Copyright (C) 2013 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 # copyright notice, this list of conditions and the following disclaimer
@@ skipping 130 matching lines @@
         'activity_logging_getter': v8_utilities.has_activity_logging(attribute, includes, 'Getter'),  # [ActivityLogging]
         'is_check_security_for_node': is_check_security_for_node,
         'is_getter_raises_exception': is_getter_raises_exception,
         'is_unforgeable': 'Unforgeable' in extended_attributes,
     })
 
     # Setter
     if not has_setter:
         return contents, includes
 
-    if v8_types.interface_type(idl_type) and not v8_types.array_type(idl_type):
-        cpp_value = 'WTF::getPtr(cppValue)'
-    else:
-        cpp_value = 'cppValue'
     contents.update({
         'v8_value_to_local_cpp_value': v8_types.v8_value_to_local_cpp_value(idl_type, attribute.extended_attributes, 'jsValue', 'cppValue', includes, 'info.GetIsolate()'),
-        'cpp_setter': 'imp->set%s(%s)' % (capitalize(cpp_name(attribute)), cpp_value),
+        'cpp_setter': setter_expression(attribute, contents),
     })
 
     return contents, includes
 
 
 def getter_expression(interface, attribute, contents, includes):
     arguments = []
     if 'Reflect' in attribute.extended_attributes:
         getter_base_name = content_attribute_getter_base_name(attribute, includes, arguments)
     else:
@@ skipping 34 matching lines @@
     arguments.append(scoped_name)
 
     idl_type = attribute.data_type
     if idl_type in CONTENT_ATTRIBUTE_GETTER_NAMES:
         return CONTENT_ATTRIBUTE_GETTER_NAMES[idl_type]
     if 'URL' in attribute.extended_attributes:
         return 'getURLAttribute'
     return 'fastGetAttribute'
 
 
+def setter_expression(attribute, contents):
+    arguments = v8_utilities.call_with_arguments(attribute, contents)
+    idl_type = attribute.data_type
+    # FIXME: should be able to eliminate WTF::getPtr in most or all cases
+    cpp_value = 'WTF::getPtr(cppValue)' if v8_types.interface_type(idl_type) and not v8_types.array_type(idl_type) else 'cppValue'
+    arguments.append(cpp_value)
+    return 'imp->set%s(%s)' % (capitalize(cpp_name(attribute)), ', '.join(arguments))
+
+
 def is_keep_alive_for_gc(attribute):
     idl_type = attribute.data_type
     extended_attributes = attribute.extended_attributes
     return (
         'KeepAttributeAliveForGC' in extended_attributes or
         # For readonly attributes, for performance reasons we keep the attribute
         # wrapper alive while the owner wrapper is alive, because the attribute
         # never changes.
         (attribute.is_read_only and
          v8_types.wrapper_type(idl_type) and
@@ skipping 51 matching lines @@
     if 'PerWorldBindings' not in attribute.extended_attributes:
         return '0'
     return '%sV8Internal::%sAttributeGetterCallbackForMainWorld' % (cpp_name(interface), attribute.name)
 
 
 def setter_callback_name_for_main_world(interface, attribute):
     if ('PerWorldBindings' not in attribute.extended_attributes or
         attribute.is_read_only):
         return '0'
     return '%sV8Internal::%sAttributeSetterCallbackForMainWorld' % (cpp_name(interface), attribute.name)