- From: Boris Zbarsky <notifications@github.com>
- Date: Mon, 07 Aug 2017 22:41:19 -0700
- To: heycam/webidl <webidl@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Tuesday, 8 August 2017 05:41:46 UTC
The comments in the `Storage` test don't quite make sense to me. `[[Set]]` there should land in https://heycam.github.io/webidl/#legacy-platform-object-set which will call the named setter regardless of what the named property algorithm says (this is purposeful, fwiw). It will not add a normal own property. What it will do is add a thing named "getItem" to the storage. But getting the list of own properties _will_ check the visibility algorithm and nor claim a "getItem" property. Hence `typeof sessionStorage.getItem` should be `"function"`, `Reflect.ownKeys(sessionStorage)` should not include `"getItem"`, etc. The bit about whether an index should be treated as a name for Storage seems to be https://github.com/heycam/webidl/issues/366 which has stalled for some reason. :( I agree compat around that stuff is poor. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/heycam/webidl/pull/402#issuecomment-320855877
Received on Tuesday, 8 August 2017 05:41:46 UTC