IDL: Use ALLOW_ONLY_INLINE_ALLOCATION() in dictionaries

Source/bindings/templates/methods.cpp

 {##############################################################################}
 {% macro generate_method(method, world_suffix) %}
 {% filter conditional(method.conditional_string) %}
 static void {{method.name}}{{method.overload_index}}Method{{world_suffix}}(const v8::FunctionCallbackInfo<v8::Value>& info)
 {
     {# Local variables #}
     {% if method.has_exception_state %}
     ExceptionState exceptionState(ExceptionState::ExecutionContext, "{{method.name}}", "{{interface_name}}", info.Holder(), info.GetIsolate());
     {% endif %}
     {# Overloaded methods have length checked during overload resolution #}
@@ skipping 215 matching lines @@
 {# [ConstructorCallWith=Document] #}
 Document& document = *toDocument(currentExecutionContext(info.GetIsolate()));
 {% endif %}
 {# Call #}
 {% if method.idl_type == 'void' %}
 {{cpp_value}};
 {% elif method.is_implemented_in_private_script %}
 {{method.cpp_type}} result{{method.cpp_type_initializer}};
 if (!{{method.cpp_value}})
     return;
+{% elif method.use_argument_for_return_value %}
+{{method.cpp_type}} cppValue;

[haraken, 2014/10/15 14:38:29]

I'd rename |cppValue| to |result| for consistency with line 242.

[bashi, 2014/10/17 00:50:05]

Done.

+{{cpp_value}};
 {% elif method.is_constructor %}
 {{method.cpp_type}} impl = {{cpp_value}};
 {% elif method.use_local_result and not method.union_arguments %}
 {{method.cpp_type}} result = {{cpp_value}};

[haraken, 2014/10/15 14:38:29]

We were trying to _always_ use local result to simplify the IDL compiler, but
what's the status of the work? (I remember we discussed a lot but I forgot the
conclusion :-)

[Jens Widell, 2014/10/15 15:39:17]

https://codereview.chromium.org/386613002/

The status is that there wasn't really a conclusion, I think, and then  nothing
more happened. I had forgotten about it, to be honest.

[bashi, 2014/10/17 00:50:05]

I had forgotten too. Maybe we can revisit this when this becomes a real problem.

[haraken, 2014/10/17 00:57:12]

The reason I asked this is that forcing a local result might make your situation
easier. Currently there are three ways to get a cpp result:

1) Use a local result.
2) Use a on-stack return value.
3) Use an output parameter to store a return value.

I just thought that removing 1) might help simplify the code. Duuno.

 {% endif %}
 {# Post-call #}
 {% if method.is_raises_exception %}
 if (exceptionState.hadException()) {
     {{throw_from_exception_state(method)}};
     return;
 }
 {% endif %}
 {# Set return value #}
 {% if method.is_constructor %}
@@ skipping 366 matching lines @@
   if method.is_per_world_bindings else '0' %}
 {% set property_attribute =
   'static_cast<v8::PropertyAttribute>(%s)' % ' | '.join(method.property_attributes)
   if method.property_attributes else 'v8::None' %}
 {% set only_exposed_to_private_script = 'V8DOMConfiguration::OnlyExposedToPrivateScript' if method.only_exposed_to_private_script else 'V8DOMConfiguration::ExposedToAllScripts' %}
 static const V8DOMConfiguration::MethodConfiguration {{method.name}}MethodConfiguration = {
     "{{method.name}}", {{method_callback}}, {{method_callback_for_main_world}}, {{method.length}}, {{only_exposed_to_private_script}},
 };
 V8DOMConfiguration::installMethod({{method.function_template}}, {{method.signature}}, {{property_attribute}}, {{method.name}}MethodConfiguration, isolate);
 {%- endmacro %}