Support FormData on WorkerGlobalScope.

Source/bindings/scripts/v8_methods.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 162 matching lines @@
         'v8_set_return_value_for_main_world': v8_set_return_value(interface.name, method, this_cpp_value, for_main_world=True),
         'v8_set_return_value': v8_set_return_value(interface.name, method, this_cpp_value),
         'v8_value_to_local_cpp_value': v8_value_to_local_cpp_value(argument, index),
     }
 
 
 ################################################################################
 # Value handling
 ################################################################################
 
-def cpp_value(interface, method, number_of_arguments):
+def cpp_value(interface, method, number_of_arguments, for_constructor=False):

[Nils Barth (inactive), 2014/04/09 02:00:27]

I don't think for_constructor is necessary:
you can just check method.is_constructor

[sof, 2014/04/09 07:31:59]

'method' is an IdlOperation/IdlAttribute/.. object, so that wouldn't quite work.
I've switched to doing a bit of type testing with .name, however - hope that
addresses the concern.

[Nils Barth (inactive), 2014/04/09 09:36:38]

Got it!
It does -- much clearer, thanks!

     def cpp_argument(argument):
         idl_type = argument.idl_type
         if idl_type.name == 'EventListener':
             if (interface.name == 'EventTarget' and
                 method.name == 'removeEventListener'):
                 # FIXME: remove this special case by moving get() into
                 # EventTarget::removeEventListener
                 return '%s.get()' % argument.name
             return argument.name
         if (idl_type.is_callback_interface or
             idl_type.name in ['NodeFilter', 'XPathNSResolver']):
             # FIXME: remove this special case
             return '%s.release()' % argument.name
         return argument.name
 
     # Truncate omitted optional arguments
     arguments = method.arguments[:number_of_arguments]
     cpp_arguments = v8_utilities.call_with_arguments(method)
     # Members of IDL partial interface definitions are implemented in C++ as
     # static member functions, which for instance members (non-static members)
     # take *impl as their first argument
     if ('PartialInterfaceImplementedAs' in method.extended_attributes and
         not method.is_static):
         cpp_arguments.append('*impl')
     cpp_arguments.extend(cpp_argument(argument) for argument in arguments)
-    this_union_arguments = method.idl_type.union_arguments
+    this_union_arguments = method.idl_type and method.idl_type.union_arguments
     if this_union_arguments:
         cpp_arguments.extend(this_union_arguments)
 
     if 'RaisesException' in method.extended_attributes:
         cpp_arguments.append('exceptionState')
 
-    cpp_method_name = v8_utilities.scoped_name(interface, method, v8_utilities.cpp_name(method))
+    if for_constructor:
+        cpp_method_name = "%s::create" % v8_utilities.cpp_name(interface)

[Nils Barth (inactive), 2014/04/09 02:00:27]

I think you can push this into v8_utilities.scoped_name:
just add:
if definition.is_constructor:
    return '%s::%s' % (cpp_name(interface), 'create')
...to it.

Also, single quotes please.

[sof, 2014/04/09 07:31:59]

For the same reasons as the above comment, attribute isn't present on this
object. Have tweaked the code a bit to allow reuse of scoped_name() though,
again consulting .name.

+    else:
+        cpp_method_name = v8_utilities.scoped_name(interface, method, v8_utilities.cpp_name(method))
     return '%s(%s)' % (cpp_method_name, ', '.join(cpp_arguments))
 
 
 def v8_set_return_value(interface_name, method, cpp_value, for_main_world=False):
     idl_type = method.idl_type
     extended_attributes = method.extended_attributes
     if idl_type.name == 'void':
         return None
 
     release = False
@@ skipping 45 matching lines @@
 
 
 def union_arguments(idl_type):
     """Return list of ['result0Enabled', 'result0', 'result1Enabled', ...] for union types, for use in setting return value"""
     return [arg
             for i in range(len(idl_type.member_types))
             for arg in ['result%sEnabled' % i, 'result%s' % i]]
 
 IdlType.union_arguments = property(lambda self: None)
 IdlUnionType.union_arguments = property(union_arguments)