Re: [heycam/webidl] Introduce a SameRealmBrandCheck extended attribute (#718)

As for usefulness, I think the spec text helps capture this. We'd like to reduce the complexity and un-polyfillable magic in some new specs we're working on.

@bzbarsky, the semantics of checking the realm seem pretty clear. What could we do to make them clearer?

I agree it's a bit tricky to use "implements" as the point at which to change this, but it is a very nice place because it covers everything. (Operations, attributes, incoming parameters, ...) Maybe we could do a quick audit of usage sites of "implements", or just have a backup where if there is no current Realm then we don't do that check.

Otherwise, I have the same issues as in https://github.com/heycam/webidl/pull/719#issuecomment-487145911, that we may want to signpost this as experimental or just make it part of being in a module, since that's a nice clean break point.

-- 
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/718#issuecomment-487147423

Received on Friday, 26 April 2019 18:01:15 UTC