On Mon Jun 19 09:36:07 2023 +0000, Jacek Caban wrote:
This test does not work in MSHTML and, as expected, uses 100 here. This looks more like a bug in jscript.dll than anything intended, so maybe we could just implement the MSHTML behavior. I was surprised that you still need a separated buffer in arguments object and it won't work for the proper implementation.
Interesting. Maybe it's intended or something makes use of it, as we already had tests exercising something similar (slightly above in the same file); I was confused at first too when I was changing it and had to make it work.
I can certainly fix it for MSHTML, but I think the current arguments buf should definitely stay, because it will be required for "strict mode" arguments object anyway, so it's not the "wrong way forward" at least (those are *always* detached from the local vars, so we could use the buf on initialization—not that I plan to implement it soon, though, just saying it will be needed at some point).
Should I attempt the MSHTML fix in this MR or the follow up? Obviously a separate commit.