[Bug 25412] Allow some recursion in NodeIterator/TreeWalker

https://www.w3.org/Bugs/Public/show_bug.cgi?id=25412

--- Comment #7 from Aryeh Gregor <ayg@aryeh.name> ---
(In reply to Boris Zbarsky from comment #6)
> Why not add a limit of 20 on event dispatch and other callback invocations? 
> Because I don't think it's needed and I don't think we need that compat hit.

What currently happens in these cases?

> > The required changes to Gecko code would be negligible.
> 
> I very much disagree, but maybe you're talking about a different change than
> the one I think you're talking about?

Change mInAcceptNode from boolean to int8_t, change "if (mInAcceptNode)" to "if
(mInAcceptNode > MAX)", change "mInAcceptNode = true" to "mInAcceptNode++",
change the AutoRestore thing to decrement instead of resetting to false. 
Probably rename mInAcceptNode to mAcceptNodeCtr or something.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.

Received on Wednesday, 23 April 2014 15:12:53 UTC