Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Layering: Implement HostEnqueuePromiseJob #4722

Closed
wants to merge 2 commits into from

Conversation

littledan
Copy link
Contributor

@littledan littledan commented Jun 20, 2019

The JavaScript specification is changing to make a HostEnqueuePromiseJob
hook to be implemented by embedders, see tc39/ecma262#1597.

This PR allows HTML to avoid its willful violation, and delete an explanation of the
motivation for this violation.

(Deleted the PR template as this change is editorial.)


/index.html ( diff )
/infrastructure.html ( diff )
/webappapis.html ( diff )

The JavaScript specification is changing to make a HostEnqueuePromiseJob
hook to be implemented by embedders. This allows HTML to avoid its willful
violation, and delete an explanation of the motivation for this violation.
Copy link
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM <3. Will tag "do not merge yet" to wait for the ecma262 PR.

@domenic domenic added the do not merge yet Pull request must not be merged per rationale in comment label Jun 21, 2019
@domenic domenic self-assigned this Jul 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge yet Pull request must not be merged per rationale in comment
Development

Successfully merging this pull request may close these issues.

2 participants