[webkit-dev] AncestorChainWalker substance and style thoughts

[Darin Adler]

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