[bindings] Support is_explicit_nullable for method arguments.

Source/bindings/templates/methods.cpp

Index: Source/bindings/templates/methods.cpp
diff --git a/Source/bindings/templates/methods.cpp b/Source/bindings/templates/methods.cpp
index fd2d2c9378652177df8337ac5adf7a77925fe24d..7aa5df857d40ccec9d801830a79d98ace82c47bc 100644
--- a/Source/bindings/templates/methods.cpp
+++ b/Source/bindings/templates/methods.cpp
@@ -170,6 +170,10 @@ if (!isUndefinedOrNull(info[{{argument.index}}]) && !info[{{argument.index}}]->I
 }
 {% endif %}{# not argument.use_permissive_dictionary_conversion #}
 {{argument.v8_value_to_local_cpp_value}};
+{% elif argument.is_explicit_nullable %}
+if (!info[{{argument.index}}]->IsNull()) {
+    {{argument.v8_value_to_local_cpp_value}};

[Jens Widell, 2015/02/24 14:29:03]

This may or may not be the only part that should be conditional. :-) (This macro
is very hard to read; we should probably refactor it.)

I thought about implementing the conditional wrapping in the
generate_arguments() where we already have similar wrapping for handling
optional arguments (with default values), but that's not really a good idea
since it would need to be duplicated then.

Other options are probably available, but I think "refactor this macro" is the
approach we ought to take before thinking much more about this (assuming this
addition turns out to be enough, at least for now.)

[Jens Widell, 2015/02/24 14:30:28]

To clarify: I think this addition here is probably fine.

+}
 {% else %}{# argument.is_nullable #}
 {{argument.v8_value_to_local_cpp_value}};
 {% endif %}{# argument.is_nullable #}