binding: Indexed properties should accompany with %ArrayProto_values%.

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 408 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)
 
-        if interface.iterable or interface.maplike or interface.setlike or 'Iterable' in extended_attributes:
+        if interface.has_indexed_elements:
+            # Window.idl in Blink has indexed properties, but the spec says
+            # Window interface doesn't have indexed properties, instead
+            # the WindowProxy exotic object has indexed properties.  Thus,
+            # Window interface must not support iterators.
+            if interface.name != 'Window':
+                has_array_iterator = True

[peria, 2016/09/15 12:06:43]

nit: has_array_iterator = (interface.name != 'Window')  ?

[Yuki, 2016/09/15 12:35:47]

Done.

+        elif interface.iterable or interface.maplike or interface.setlike or 'Iterable' in extended_attributes:
             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 961 matching lines @@
     extended_attributes = deleter.extended_attributes
     is_call_with_script_state = v8_utilities.has_extended_attribute_value(deleter, 'CallWith', 'ScriptState')
     is_ce_reactions = 'CEReactions' in extended_attributes
     return {
         'is_call_with_script_state': is_call_with_script_state,
         'is_ce_reactions': is_ce_reactions,
         'is_custom': 'Custom' in extended_attributes,
         'is_raises_exception': 'RaisesException' in extended_attributes,
         'name': cpp_name(deleter),
     }