[webkit-dev] AncestorChainWalker substance and style thoughts

Darin Adler darin at apple.com
Mon Feb 18 18:38:33 PST 2013

On Feb 18, 2013, at 6:28 PM, Ryosuke Niwa <rniwa at webkit.org> wrote:

> I had the same question when I found this class today but when I tried to turn it into a function, I realized that each caller of this function now needs to maintain two variables instead of one. Since those two variables (node and distributedNode) need to be in a consistent state, I've thought of creating a state object but then it seemed silly to create a state object only and a free function. It's much more natural for it be just a "walker" class.

Does it? Is it really necessary to have the distributed node state or is that just performance optimization?

-- Darin

More information about the webkit-dev mailing list