[Webkit-unassigned] [Bug 126684] Switch HTMLTableRowsCollection from Traversal<> to iterators

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jan 9 07:47:36 PST 2014


https://bugs.webkit.org/show_bug.cgi?id=126684





--- Comment #6 from Darin Adler <darin at apple.com>  2014-01-09 07:45:18 PST ---
(From update of attachment 220705)
View in context: https://bugs.webkit.org/attachment.cgi?id=220705&action=review

>>> Source/WebCore/dom/ElementChildIterator.h:160
>>> +        return end();
>> 
>> If this element is a child of m_parent, just not the correct type, I would expect this function to return the next child of the correct type if any, not always end().
>> 
>> A way to avoid this problem entirely would be to make the argument be ElementType& instead of const Element&.
> 
> I guess I’m wrong. We are using end() here as a sort of null value.
> 
> But I do think that we should have a find that takes the specific element type to avoid the runtime isElementOfType check.

I guess I really don’t like find for the name of this. It sounds like something that iterates the children, when really it’s just a sort of alternate iterator constructor.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list