[Bug 25412] Allow some recursion in NodeIterator/TreeWalker

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

--- Comment #36 from Aryeh Gregor <ayg@aryeh.name> ---
To be more precise: the current spec specifies no special recursion limit,
which is closer to a limit of 10000 or 100 (Edge/Blink) than 1 (Gecko).

Or, if you prefer: the current spec matches Edge and Blink for reasonable code,
where code that recurses more than 100 levels is classified as "unreasonable".

However you phrase it exactly, I still don't see any argument for prohibiting
recursion here entirely.

You are correct that all UAs do have some sort of recursion limit here that
seems to be lower than their generic JavaScript stack limit, based on comparing
these two tests:

http://software.hixie.ch/utilities/js/live-dom-viewer/?saved=2957
http://software.hixie.ch/utilities/js/live-dom-viewer/?saved=4387

Apparently such an extra limit is necessary, although I'm not sure why.  If we
wanted to spec it, we could, but I'm not sure it's necessary unless you think
the difference between 100 and 10000 is going to be significant on real-world
pages.  Generally I think we don't bother speccing this sort of arbitrary cap,
although maybe we should?

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

Received on Monday, 22 August 2016 14:17:33 UTC