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

Reland "[blink] Fix top-level-await error reporting" #25530

Closed
wants to merge 1 commit into from

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Sep 15, 2020

This is a reland of 077d63da3268e6761d20c56409d37c57d37f1685

Original change's description:

[blink] Fix top-level-await error reporting

  • Enable tests with TLA enabled
  • Fix top-level error reporting with TLA

Based on the discussions on the spec, we fire error events on rejection
of the result promise:
whatwg/html#4352 (comment)

Bug: 1022182, 1096455, 1127215, v8:9344
Change-Id: I16e83cb4e279c1e44be7fa70a51a103ee94aacc3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2228881
Commit-Queue: Camillo Bruni <cbruni@chromium.org>
Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Dominic Farolino <dom@chromium.org>
Reviewed-by: Domenic Denicola <domenic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#806593}

Bug: 1022182
Bug: 1096455
Bug: 1127215
Bug: v8:9344
Bug: 1128296
Change-Id: Icecaccf34efc4354e0faac362da6f6cbdd49f50d
Reviewed-on: https://chromium-review.googlesource.com/2410075
WPT-Export-Revision: c726631903c0284fdc0088fdb51608dd99c79f9e

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-2410075 branch 2 times, most recently from 9b18f41 to 306a5bf Compare September 15, 2020 10:30
This is a reland of 077d63da3268e6761d20c56409d37c57d37f1685

Original change's description:
> [blink] Fix top-level-await error reporting
>
> - Enable tests with TLA enabled
> - Fix top-level error reporting with TLA
>
> Based on the discussions on the spec, we fire error events on rejection
> of the result promise:
> whatwg/html#4352 (comment)
>
> Bug: 1022182, 1096455, 1127215, v8:9344
> Change-Id: I16e83cb4e279c1e44be7fa70a51a103ee94aacc3
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2228881
> Commit-Queue: Camillo Bruni <cbruni@chromium.org>
> Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
> Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
> Reviewed-by: Dominic Farolino <dom@chromium.org>
> Reviewed-by: Domenic Denicola <domenic@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#806593}

Bug: 1022182
Bug: 1096455
Bug: 1127215
Bug: v8:9344
Bug: 1128296
Change-Id: Icecaccf34efc4354e0faac362da6f6cbdd49f50d
@chromium-wpt-export-bot
Copy link
Collaborator Author

Close this PR because the Chromium CL does not have exportable changes.

@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-2410075 branch October 23, 2020 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants