[DeprecateAs] if call addEventListener or removeEventListener without enough arguments

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 #}
     {% if method.number_of_required_arguments and not method.overload_index %}
     if (UNLIKELY(info.Length() < {{method.number_of_required_arguments}})) {
         {{throw_minimum_arity_type_error(method, method.number_of_required_arguments)}};
         return;
     }
     {% endif %}
     {% if not method.is_static %}
     {{cpp_class}}* impl = {{v8_class}}::toNative(info.Holder());
     {% endif %}
     {% if method.is_custom_element_callbacks %}
     CustomElementCallbackDispatcher::CallbackDeliveryScope deliveryScope;
     {% endif %}
     {# Security checks #}
-    {# FIXME: change to method.is_check_security_for_window #}
+    {# FIXME: merge to method.is_check_security_for_frame http://crbug.com/383699 #}
     {% if interface_name == 'EventTarget' %}
     if (LocalDOMWindow* window = impl->toDOMWindow()) {
         if (!BindingSecurity::shouldAllowAccessToFrame(info.GetIsolate(), window->frame(), exceptionState)) {
             {{throw_from_exception_state(method)}};
             return;
         }
         if (!window->document())
             return;
     }
     {% elif method.is_check_security_for_frame %}
@@ skipping 216 matching lines @@
 {% endif %}
 {# Set return value #}
 {% if method.is_constructor %}
 {{generate_constructor_wrapper(method)}}
 {%- elif method.union_arguments %}
 {{union_type_method_call_and_set_return_value(method)}}
 {%- elif v8_set_return_value %}
 {{v8_set_return_value}};
 {%- endif %}{# None for void #}
 {# Post-set #}
-{% if interface_name == 'EventTarget' and method.name in ('addEventListener',
-                                                          'removeEventListener') %}
+{# Arguments length check needed to skip action on legacy calls without enough
+   arguments. http://crbug.com/353484 #}
+{% if interface_name == 'EventTarget' and
+      method.name in ('addEventListener', 'removeEventListener') and
+      method.arguments|length >= 2 %}
 {% set hidden_dependency_action = 'addHiddenValueToArray'
        if method.name == 'addEventListener' else 'removeHiddenValueFromArray' %}
-{# Length check needed to skip action on legacy calls without enough arguments.
-   http://crbug.com/353484 #}
-if (info.Length() >= 2 && listener && !impl->toNode())
+if (listener && !impl->toNode())
     {{hidden_dependency_action}}(info.Holder(), info[1], {{v8_class}}::eventListenerCacheIndex, info.GetIsolate());
 {% endif %}
 {% endmacro %}
 
 
 {######################################}
 {% macro union_type_method_call_and_set_return_value(method) %}
 {% for cpp_type in method.cpp_type %}
 bool result{{loop.index0}}Enabled = false;
 {{cpp_type}} result{{loop.index0}};
@@ skipping 284 matching lines @@
     {% endif %}
     {{constructor.cpp_type}} impl = {{constructor.cpp_value}};
     {% if is_constructor_raises_exception %}
     if (exceptionState.throwIfNeeded())
         return;
     {% endif %}
 
     {{generate_constructor_wrapper(constructor) | indent}}
 }
 {% endmacro %}