Re: [heycam/webidl] Added a definition of {{StringContext}} extended attribute. (#841)

I agree with @domenic here, we're avoiding type unions to avoid branching in other spec algos, and centralize the behavior in WebIDL (to define the extended attribute and callouts) and HTML (to implement the TT-specific check). 

The current Blink implementation follows the PR above ("option 1 + before" from https://github.com/heycam/webidl/pull/841#issuecomment-586354298), as we need the original type information when checking. In our impl piping through the context ended up being quite simple, and we get the benefits of removing many custom changes at the actual setters (the latter would be possible with other options too, as long as we don't choose the "type unions" approach). I think the decision process for choosing option 1 is neatly summarized in https://github.com/heycam/webidl/pull/841#issuecomment-586421175. 

-- 
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/841#issuecomment-595756809