On Tue Dec 3 19:17:02 2024 +0000, Jacek Caban wrote:
To implement them properly we'd need overrides like with the other
volatile objects (localStorage for example), which needs a revamp on jscript, so for now just add the tests. I guess what we need is an additional "don't enum" flag which would mean "don't enumerate even if marked as enumerable". It shouldn't be too hard to add, I wouldn't call it a revamp, but it's definitely fine to skip for now.
There's more to it than that. defineProperty over it will define normal prop and override it, but upon delete, the original one is revealed (with tests). I suspect it's using the same mechanism the other volatile lookups are (for window, document, localStorage, etc), and it's looked in a special way after the prototypes are. Though I haven't added tests for that yet, and anyway it's out of this scope of this MR.