Rename LiveNodeLists / HTMLCollections's nodeMatches() to elementMatches() for clarity as the argument is an Element.
Created attachment 238148 [details] Patch
Comment on attachment 238148 [details] Patch Clearing flags on attachment: 238148 Committed r173649: <http://trac.webkit.org/changeset/173649>
All reviewed patches have been landed. Closing bug.
Comment on attachment 238148 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=238148&action=review > Source/WebCore/dom/ClassNodeList.h:49 > + virtual bool elementMatches(Element*) const override; Since we are touching every single call site, it really seems we’d want to change this to Element&. Too late!
Comment on attachment 238148 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=238148&action=review >> Source/WebCore/dom/ClassNodeList.h:49 >> + virtual bool elementMatches(Element*) const override; > > Since we are touching every single call site, it really seems we’d want to change this to Element&. Too late! Right, sorry about that. I'll follow-up on this.