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

@bzbarsky Thanks! I've updated the comments in the file, and it does make things clearer. To summarize, for `Storage`:

- Firefox and Edge's setters incorrectly ignore symbols
- Chrome's getter incorrectly gets the shadowed prototype property even when it should not be visible
- Edge's getter correctly handles shadowed properties, but still makes them visible through `Reflect.ownKeys()`.
- Chrome enumerates symbols in chronological order instead of last.
- Chrome and Firefox put integer index properties first, even when it is really a named property and should be enumerated chronologically. *Within* integer index properties, Chrome then sorts *them* numerically. Firefox sorts them chronologically.

Do you have any comments regarding this PR itself though?

-- 
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-320892495