Create fewer local v8::TryCatch objects in generated bindings code

Source/bindings/scripts/v8_interface.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 446 matching lines @@
         return type_check
     return None
 
 
 ################################################################################
 # Constructors
 ################################################################################
 
 # [Constructor]
 def generate_constructor(interface, constructor):
+    generated_arguments = [constructor_argument(interface, constructor, argument, index)
+                           for index, argument in enumerate(constructor.arguments)]
+
     return {
         'argument_list': constructor_argument_list(interface, constructor),
-        'arguments': [constructor_argument(interface, constructor, argument, index)
-                      for index, argument in enumerate(constructor.arguments)],
+        'arguments': generated_arguments,
+        'arguments_need_try_catch': any(
+            v8_methods.argument_needs_try_catch(argument) for argument in generated_arguments),

[Nils Barth (inactive), 2014/05/07 03:02:19]

Could you add one more linebreak here?
(So for ... on separate line.)

         'cpp_type': cpp_template_type(
             cpp_ptr_type('RefPtr', 'RawPtr', gc_type(interface)),
             cpp_name(interface)),
         'has_exception_state':
             # [RaisesException=Constructor]
             interface.extended_attributes.get('RaisesException') == 'Constructor' or
             any(argument for argument in constructor.arguments
                 if argument.idl_type.name == 'SerializedScriptValue' or
                    argument.idl_type.is_integer_type),
         'is_constructor': True,
@@ skipping 16 matching lines @@
     arguments.extend([argument.name for argument in constructor.arguments])
 
     # [RaisesException=Constructor]
     if interface.extended_attributes.get('RaisesException') == 'Constructor':
         arguments.append('exceptionState')
 
     return arguments
 
 
 def constructor_argument(interface, constructor, argument, index):
+    extended_attributes = argument.extended_attributes
     idl_type = argument.idl_type
+    is_variadic_wrapper_type = argument.is_variadic and idl_type.is_wrapper_type
+
     return {
+        'cpp_type': idl_type.cpp_type_args(extended_attributes=extended_attributes,
+                                           used_as_argument=not is_variadic_wrapper_type,
+                                           used_in_cpp_sequence=is_variadic_wrapper_type),
         'cpp_value':
             v8_methods.cpp_value(interface, constructor, index),
         'has_default': 'Default' in argument.extended_attributes,
         'has_legacy_overload_string': False,  # Required for overload resolution
         # Dictionary is special-cased, but arrays and sequences shouldn't be
         'idl_type': not idl_type.array_or_sequence_type and idl_type.base_type,
         'idl_type_object': idl_type,
         'index': index,
         'is_optional': argument.is_optional,
         'name': argument.name,
@@ skipping 228 matching lines @@
         deleter = next(
             method
             for method in interface.operations
             if ('deleter' in method.specials and
                 len(method.arguments) == 1 and
                 str(method.arguments[0].idl_type) == 'DOMString'))
     except StopIteration:
         return None
 
     return property_deleter(deleter)