Chromium Code Reviews| Index: Source/bindings/scripts/v8_methods.py |
| diff --git a/Source/bindings/scripts/v8_methods.py b/Source/bindings/scripts/v8_methods.py |
| index f4253981e4a1c54c1e5af8d2c26bf4a0446f0e82..0e07f6c76c772273f775fa26fd0b47b6ec35fbda 100644 |
| --- a/Source/bindings/scripts/v8_methods.py |
| +++ b/Source/bindings/scripts/v8_methods.py |
| @@ -211,10 +211,15 @@ def argument_context(interface, method, argument, index): |
| raise Exception('Private scripts supports only primitive types and DOM wrappers.') |
| set_default_value = argument.set_default_value |
| + this_cpp_type = None |
| + if idl_type.is_explicit_nullable and not argument.is_variadic: |
|
Jens Widell
2015/03/02 08:42:17
This is not quite what I meant. I meant to do
t
|
| + this_cpp_type = v8_types.cpp_template_type('Nullable', idl_type.cpp_type) |
| + else: |
| + this_cpp_type = idl_type.cpp_type_args(extended_attributes=extended_attributes, |
| + raw_type=True, |
| + used_as_variadic_argument=argument.is_variadic) |
| return { |
| - 'cpp_type': idl_type.cpp_type_args(extended_attributes=extended_attributes, |
| - raw_type=True, |
| - used_as_variadic_argument=argument.is_variadic), |
| + 'cpp_type': this_cpp_type, |
| 'cpp_value': this_cpp_value, |
| # FIXME: check that the default value's type is compatible with the argument's |
| 'set_default_value': set_default_value, |
| @@ -231,6 +236,7 @@ def argument_context(interface, method, argument, index): |
| 'is_callback_interface': idl_type.is_callback_interface, |
| # FIXME: Remove generic 'Dictionary' special-casing |
| 'is_dictionary': idl_type.is_dictionary or idl_type.base_type == 'Dictionary', |
| + 'is_explicit_nullable': idl_type.is_explicit_nullable, |
| 'is_nullable': idl_type.is_nullable, |
| 'is_optional': argument.is_optional, |
| 'is_variadic': argument.is_variadic, |