Support per-overload [RuntimeEnabled] extended attribute

Source/bindings/templates/interface.cpp

Index: Source/bindings/templates/interface.cpp
diff --git a/Source/bindings/templates/interface.cpp b/Source/bindings/templates/interface.cpp
index 7078e454decd8f034d5359708d056b968aef5bfb..81992ddc87d3f25b053761712ac162beaadc0de4 100644
--- a/Source/bindings/templates/interface.cpp
+++ b/Source/bindings/templates/interface.cpp
@@ -986,11 +986,14 @@ static void configure{{v8_class}}Template(v8::Handle<v8::FunctionTemplate> funct
     {# Needed for legacy support of document.all #}
     functionTemplate->InstanceTemplate()->MarkAsUndetectable();
     {% endif %}
-    {% for method in methods if not method.do_not_check_signature %}
+    {% for method in methods %}
     {# install_custom_signature #}
-    {% if not method.overload_index or method.overload_index == 1 %}
+    {% if not(method.overloads.do_not_check_signature_all if method.overloads else
+              (method.overload_index or method.do_not_check_signature)) %}
     {# For overloaded methods, only generate one accessor #}
     {% filter conditional(method.conditional_string) %}
+    {% filter runtime_enabled(method.overloads.runtime_enabled_function_all if method.overloads else

[Jens Widell, 2014/06/04 11:20:28]

This is wrong if all overloads are runtime enabled, but by more than one feature
flag. You could end up with a method registered that supports zero overloads and
thus always fails when called. (And traces of experimental features could
potentially be shipped in the form of useless stub methods by mistake.)

Don't know how likely it is that this would ever happen. Should we add a FIXME
somewhere in v8_interface.py and raise an exception if such a set of overloads
is ever defined?

[Jens Widell, 2014/06/05 08:51:51]

It occurs to me that the "Function.length is dynamic" compile-time check that
issue 296403007 adds is such a check, somewhat unintentionally.

+                              method.runtime_enabled_function) %}
     {% if method.is_do_not_check_security %}
     {% if method.is_per_world_bindings %}
     if (DOMWrapperWorld::current(isolate).isMainWorld()) {
@@ -1004,21 +1007,16 @@ static void configure{{v8_class}}Template(v8::Handle<v8::FunctionTemplate> funct
     {% else %}{# is_do_not_check_security #}
     {% if method.is_per_world_bindings %}
     if (DOMWrapperWorld::current(isolate).isMainWorld()) {
-        {% filter runtime_enabled(method.runtime_enabled_function) %}
         {{install_custom_signature(method, 'ForMainWorld')}}
-        {% endfilter %}
     } else {
-        {% filter runtime_enabled(method.runtime_enabled_function) %}
         {{install_custom_signature(method)}}
-        {% endfilter %}
     }
     {% else %}
-    {% filter runtime_enabled(method.runtime_enabled_function) %}
     {{install_custom_signature(method)}}
-    {% endfilter %}
     {% endif %}
     {% endif %}{# is_do_not_check_security #}
-    {% endfilter %}
+    {% endfilter %}{# runtime_enabled() #}
+    {% endfilter %}{# conditional() #}
     {% endif %}{# install_custom_signature #}
     {% endfor %}
     {% for attribute in attributes if attribute.is_static %}