Skip to content
This repository has been archived by the owner on Jan 20, 2020. It is now read-only.

Normative: Initial spec draft #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

littledan
Copy link

@littledan littledan commented Jan 7, 2018

This spec draft is in line with issue #2 with @allenwb's suggestion for
where the brand check function should go. Probably not ready for merging
now, but I thought it might help to have this kind of concreteness as a
strawman, to focus further discussion if this is a direction the champions
want to go.

This spec draft is in line with issue mikewest#2 with @domenic's suggestion for
where the brand check function should go.
@domenic
Copy link

domenic commented Jan 7, 2018

Wait, my suggestion was that it should be a getter on the template object, not as a static on Array. #2 (comment)

@littledan
Copy link
Author

littledan commented Jan 7, 2018

Oops, well, this uses @allenwb's suggestion!

I agree with much of Allen's reasoning there (even if the mechanism for disabling eval will probably be different from the mechanism for locking down Array.isTemplateLiteral on the web for one).

@littledan
Copy link
Author

@ajklein @mikewest What do you think, among the two alternatives proposed by @allenwb and @domenic for how the surface API should be exposed here? Or, should we not even include it for JavaScript programs and just make the internal slot available to embedders?

@ajklein
Copy link

ajklein commented Mar 5, 2018

@littledan API concerns for this proposal haven't yet bubbled to the surface, for me at least, so I don't have an opinion to offer.

There's some ongoing (offline) discussion about how well this proposal meets the intended Google use-cases, and those need to be resolved before further work is devoted to this particular proposal.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants