[Webkit-unassigned] [Bug 120358] Remove branch from DescendantIteratorAdapter::begin() when invoking for ContainerNode

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Aug 27 09:26:17 PDT 2013


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





--- Comment #5 from Darin Adler <darin at apple.com>  2013-08-27 09:25:41 PST ---
(From update of attachment 209771)
View in context: https://bugs.webkit.org/attachment.cgi?id=209771&action=review

> Source/WebCore/dom/DescendantIterator.h:41
> -    DescendantIterator(const ContainerNode* root);
> -    DescendantIterator(const ContainerNode* root, ElementType* current);
> +    DescendantIterator(const Node* root);
> +    DescendantIterator(const Node* root, ElementType* current);

Why is this a good change? I understand it’s only used for equality comparison, but are there really callers that need to use this with a Node*?

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