[bindings] add support for integer-indexed @@iterator

third_party/WebKit/Source/bindings/scripts/v8_interface.py

 # Copyright (C) 2013 Google Inc. All rights reserved.
 # coding=utf-8
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
@@ skipping 330 matching lines @@
         argument = IdlArgument(interface.idl_name)
         argument.idl_type = idl_type
         argument.name = name
         argument.is_optional = is_optional
         if extended_attributes:
             argument.extended_attributes.update(extended_attributes)
         return argument
 
     # [Iterable], iterable<>, maplike<> and setlike<>
     iterator_method = None
+    has_array_iterator = False
+
     # FIXME: support Iterable in partial interfaces. However, we don't
     # need to support iterator overloads between interface and
     # partial interface definitions.
     # http://heycam.github.io/webidl/#idl-overloading
     if (not interface.is_partial
         and (interface.iterable or interface.maplike or interface.setlike
-             or 'Iterable' in extended_attributes)):
+             or interface.has_indexed_elements or 'Iterable' in extended_attributes)):
 
         used_extended_attributes = {}
 
         if interface.iterable:
             used_extended_attributes.update(interface.iterable.extended_attributes)
         elif interface.maplike:
             used_extended_attributes.update(interface.maplike.extended_attributes)
         elif interface.setlike:
             used_extended_attributes.update(interface.setlike.extended_attributes)
 
@@ skipping 12 matching lines @@
             'CallWith': ['ScriptState', 'ThisValue'],
         })
 
         def generated_iterator_method(name, implemented_as=None):
             return generated_method(
                 return_type=IdlType('Iterator'),
                 name=name,
                 extended_attributes=used_extended_attributes,
                 implemented_as=implemented_as)
 
-        iterator_method = generated_iterator_method('iterator', implemented_as='iterator')
+        if (interface.iterable or interface.maplike or interface.setlike or 'Iterable' in extended_attributes):

[Jens Widell, 2015/10/05 05:57:11]

Minor nit: in Python you typically don't have parentheses here. (Only if you
break the expression up over several lines or something like that.)

+            iterator_method = generated_iterator_method('iterator', implemented_as='iterator')
+        elif interface.has_indexed_elements:
+            has_array_iterator = True
 
         if interface.iterable or interface.maplike or interface.setlike:
             implicit_methods = [
                 generated_iterator_method('keys'),
                 generated_iterator_method('values'),
                 generated_iterator_method('entries'),
 
                 # void forEach(Function callback, [Default=Undefined] optional any thisArg)
                 generated_method(IdlType('void'), 'forEach',
                                  arguments=[generated_argument(IdlType('Function'), 'callback'),
@@ skipping 152 matching lines @@
 
     context.update({
         'conditionally_enabled_methods': conditionally_enabled_methods,
         'custom_registration_methods': custom_registration_methods,
         'has_origin_safe_method_setter': any(
             method['is_check_security_for_frame'] and not method['is_read_only']
             for method in methods),
         'has_private_script': any(attribute['is_implemented_in_private_script'] for attribute in attributes) or
             any(method['is_implemented_in_private_script'] for method in methods),
         'iterator_method': iterator_method,
+        'has_array_iterator': has_array_iterator,
         'method_configuration_methods': method_configuration_methods,
         'methods': methods,
     })
 
     # Conditionally enabled members
     has_conditional_attributes_on_instance = any(
         attribute['exposed_test'] and attribute['on_instance']
         for attribute in attributes)
     has_conditional_attributes_on_prototype = any(
         attribute['exposed_test'] and attribute['on_prototype']
@@ skipping 794 matching lines @@
 
     extended_attributes = deleter.extended_attributes
     idl_type = deleter.idl_type
     is_call_with_script_state = v8_utilities.has_extended_attribute_value(deleter, 'CallWith', 'ScriptState')
     return {
         'is_call_with_script_state': is_call_with_script_state,
         'is_custom': 'Custom' in extended_attributes,
         'is_raises_exception': 'RaisesException' in extended_attributes,
         'name': cpp_name(deleter),
     }