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

PerformPromiseThen needs to set [[PromiseIsHandled]] unconditionally #383

Closed
domenic opened this Issue Feb 12, 2016 · 0 comments

Comments

Projects
None yet
1 participant
@domenic
Member

domenic commented Feb 12, 2016

https://tc39.github.io/ecma262/#sec-performpromisethen

Step 9.e. should actually be step 10. Once a promise is .thened, it becomes handled, and its responsibilities regarding unhandled rejections pass along to its derived promise.

I can PR this later today if nobody else gets to it.

Found by @tschneidereit in #jslang

domenic added a commit to domenic/ecma262 that referenced this issue Feb 12, 2016

Layering: fix unhandled rejection tracking in PerformPromiseThen
Fixes tc39#383. then-ing a promise should always mark it as handled, not
just when the promise is already rejected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment