Following the removal of the ability to call collections bug 67579, only HTMLAllCollection (document.all) needs the nextNamedItem() mechanism to iterate over all the named items in the collection. Assuming that we don't intend to reintroduce the removed functionality, we could move this complexity to HTMLAllCollection and simplify both HTMLCollection and HTMLFormCollection.
Created attachment 114416 [details] Proposed patch
Comment on attachment 114416 [details] Proposed patch Attachment 114416 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/10394182
(In reply to comment #2) > (From update of attachment 114416 [details]) > Attachment 114416 [details] did not pass chromium-ews (chromium-xvfb): > Output: http://queues.webkit.org/results/10394182 D'oh, I see Chromium still allows calling any collection. Adding some Google guys to see if they're interested in taking the same direction with the V8 bindings.
Comment on attachment 114416 [details] Proposed patch Removing r?, since this obviously can't land with V8 bindings still using HTMLCollection::nextNamedItem().
I am personally too off the spec related things to provide any meaningful judgement on this. But if there is a consensus it's a good idea to experiment with, I think all WebKit-based browsers should support this. Adam, Eric, WDYT? Should we loop in someone else, say, Dmitry? (In reply to comment #4) > (From update of attachment 114416 [details]) > Removing r?, since this obviously can't land with V8 bindings still using HTMLCollection::nextNamedItem().
IMHO, V8 should match JSC's new behavior. The callability of these interfaces is just for compatibility.
(In reply to comment #6) > IMHO, V8 should match JSC's new behavior. The callability of these interfaces is just for compatibility. Righty-o, I filed bug 72045 to remove this from the V8 bindings. :)
Created attachment 114696 [details] Same patch rebased. Okay, let's try again now that calling HTMLCollections is no longer possible via V8 bindings.
Comment on attachment 114696 [details] Same patch rebased. r=me
Committed r100090: <http://trac.webkit.org/changeset/100090>