Bug 71987 – Only HTMLAllCollection needs stateful named item traversal.

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Andreas Kling]

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.

[Andreas Kling]

Created attachment 114416 [details]
Proposed patch

[WebKit Review Bot]

Comment on attachment 114416 [details]
Proposed patch

Attachment 114416 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/10394182

[Andreas Kling]

(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.

[Andreas Kling]

Comment on attachment 114416 [details]
Proposed patch

Removing r?, since this obviously can't land with V8 bindings still using HTMLCollection::nextNamedItem().

[anton muhin]

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().

[Adam Barth]

IMHO, V8 should match JSC's new behavior.  The callability of these interfaces is just for compatibility.

[Andreas Kling]

(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. :)

[Andreas Kling]

Created attachment 114696 [details]
Same patch rebased.

Okay, let's try again now that calling HTMLCollections is no longer possible via V8 bindings.

[Antti Koivisto]

Comment on attachment 114696 [details]
Same patch rebased.

r=me

[Andreas Kling]

Committed r100090: <http://trac.webkit.org/changeset/100090>