Re: [heycam/webidl] Define [[OwnPropertyKeys]] of legacy platform objects (#402)

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