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

Mark DOMException as serializable #732

Merged
merged 7 commits into from
Jul 3, 2019
Merged

Mark DOMException as serializable #732

merged 7 commits into from
Jul 3, 2019

Conversation

yutakahirano
Copy link
Member

@yutakahirano yutakahirano commented May 30, 2019

Mark DOMException as serialiable

Also define serialization and deserialization steps.
tests: web-platform-tests/wpt#17095

Fixes #729


Preview | Diff

@yutakahirano yutakahirano marked this pull request as ready for review May 30, 2019 08:27
Also define serialization and deserialization steps.
tests: web-platform-tests/wpt#17095

Fixes #729.
@yutakahirano
Copy link
Member Author

PTAL

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 with nit. I guess we should wait for multi-implementer interest over in whatwg/html#4268. (/cc @jorendorff)

index.bs Outdated Show resolved Hide resolved
@bzbarsky
Copy link
Collaborator

Note the problem described in whatwg/html#4268 (comment)

Co-Authored-By: Domenic Denicola <[email protected]>
index.bs Outdated Show resolved Hide resolved
@yutakahirano
Copy link
Member Author

Added sentences for .stack.

index.bs Show resolved Hide resolved
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.

Still LGTM.

domenic pushed a commit to whatwg/html that referenced this pull request Jul 3, 2019
Define serialization and deserialization steps for all built-in
NativeError types from the JavaScript specification. This change doesn't
cover DOMExceptions, which are covered by
whatwg/webidl#732.

Fixes #4268.

Tests:
* web-platform-tests/wpt#17095
* web-platform-tests/wpt#17159
@domenic domenic merged commit f23ef42 into whatwg:master Jul 3, 2019
annevk pushed a commit to web-platform-tests/wpt that referenced this pull request Jul 9, 2019
domenic added a commit to web-platform-tests/wpt that referenced this pull request Jul 9, 2019
foolip pushed a commit to web-platform-tests/wpt that referenced this pull request Jul 10, 2019
aarongable pushed a commit to chromium/chromium that referenced this pull request Jul 18, 2019
Make DOMException serializable.
We don't support the stack property due to some technical difficulties.

Spec: whatwg/webidl#732
Intent-to-Ship: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/f8JngIi8qYs

Bug: 970079
Change-Id: I909bb5f32a79baac19108bc18eaf91eb597a8c85
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1697462
Commit-Queue: Yutaka Hirano <[email protected]>
Reviewed-by: Jeremy Roman <[email protected]>
Reviewed-by: Yuki Shiino <[email protected]>
Cr-Commit-Position: refs/heads/master@{#678689}
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Jul 24, 2019
…tacks, a=testonly

Automatic update from web-platform-tests
Add optional test for cloning of error stacks

Supplements web-platform-tests/wpt#17095.
Follows whatwg/html#4665 and whatwg/webidl#732.
--

wpt-commits: 1da6bed5d8c4c38200383b86928b7be68bfb87da
wpt-pr: 17159


--HG--
rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/echo-iframe.html => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/resources/echo-iframe.html
rename : testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/resources/echo-worker.js => testing/web-platform/tests/html/infrastructure/safe-passing-of-structured-data/resources/echo-worker.js
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Jul 24, 2019
…=testonly

Automatic update from web-platform-tests
Add structured clone tests for errors

This is tests for spec changes.
 - whatwg/html#4665
 - whatwg/webidl#732

--
Add tests for cloning from another realm

--
Test some stuff before cloning

--

wpt-commits: 1b4fbe0c7049d89fd805dc157f68d82616f5d203, efc79eef01ffa65273ff01736d989ff5fa79f18d, 84af6c875d378944b39d895acdcfc170736b2d3d
wpt-pr: 17095
xeonchen pushed a commit to xeonchen/gecko that referenced this pull request Jul 25, 2019
…tacks, a=testonly

Automatic update from web-platform-tests
Add optional test for cloning of error stacks

Supplements web-platform-tests/wpt#17095.
Follows whatwg/html#4665 and whatwg/webidl#732.
--

wpt-commits: 1da6bed5d8c4c38200383b86928b7be68bfb87da
wpt-pr: 17159
xeonchen pushed a commit to xeonchen/gecko that referenced this pull request Jul 25, 2019
…=testonly

Automatic update from web-platform-tests
Add structured clone tests for errors

This is tests for spec changes.
 - whatwg/html#4665
 - whatwg/webidl#732

--
Add tests for cloning from another realm

--
Test some stuff before cloning

--

wpt-commits: 1b4fbe0c7049d89fd805dc157f68d82616f5d203, efc79eef01ffa65273ff01736d989ff5fa79f18d, 84af6c875d378944b39d895acdcfc170736b2d3d
wpt-pr: 17095
natechapin pushed a commit to natechapin/wpt that referenced this pull request Aug 23, 2019
natechapin pushed a commit to natechapin/wpt that referenced this pull request Aug 23, 2019
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Oct 4, 2019
…tacks, a=testonly

Automatic update from web-platform-tests
Add optional test for cloning of error stacks

Supplements web-platform-tests/wpt#17095.
Follows whatwg/html#4665 and whatwg/webidl#732.
--

wpt-commits: 1da6bed5d8c4c38200383b86928b7be68bfb87da
wpt-pr: 17159

UltraBlame original commit: d0647ab7df5d71c33b5c35b8b4ce69940975639e
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Oct 4, 2019
…=testonly

Automatic update from web-platform-tests
Add structured clone tests for errors

This is tests for spec changes.
 - whatwg/html#4665
 - whatwg/webidl#732

--
Add tests for cloning from another realm

--
Test some stuff before cloning

--

wpt-commits: 1b4fbe0c7049d89fd805dc157f68d82616f5d203, efc79eef01ffa65273ff01736d989ff5fa79f18d, 84af6c875d378944b39d895acdcfc170736b2d3d
wpt-pr: 17095

UltraBlame original commit: 696cf52f476f2f644137d19d9b9c4ec4dc400731
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 4, 2019
…tacks, a=testonly

Automatic update from web-platform-tests
Add optional test for cloning of error stacks

Supplements web-platform-tests/wpt#17095.
Follows whatwg/html#4665 and whatwg/webidl#732.
--

wpt-commits: 1da6bed5d8c4c38200383b86928b7be68bfb87da
wpt-pr: 17159

UltraBlame original commit: d0647ab7df5d71c33b5c35b8b4ce69940975639e
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 4, 2019
…=testonly

Automatic update from web-platform-tests
Add structured clone tests for errors

This is tests for spec changes.
 - whatwg/html#4665
 - whatwg/webidl#732

--
Add tests for cloning from another realm

--
Test some stuff before cloning

--

wpt-commits: 1b4fbe0c7049d89fd805dc157f68d82616f5d203, efc79eef01ffa65273ff01736d989ff5fa79f18d, 84af6c875d378944b39d895acdcfc170736b2d3d
wpt-pr: 17095

UltraBlame original commit: 696cf52f476f2f644137d19d9b9c4ec4dc400731
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 4, 2019
…tacks, a=testonly

Automatic update from web-platform-tests
Add optional test for cloning of error stacks

Supplements web-platform-tests/wpt#17095.
Follows whatwg/html#4665 and whatwg/webidl#732.
--

wpt-commits: 1da6bed5d8c4c38200383b86928b7be68bfb87da
wpt-pr: 17159

UltraBlame original commit: d0647ab7df5d71c33b5c35b8b4ce69940975639e
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 4, 2019
…=testonly

Automatic update from web-platform-tests
Add structured clone tests for errors

This is tests for spec changes.
 - whatwg/html#4665
 - whatwg/webidl#732

--
Add tests for cloning from another realm

--
Test some stuff before cloning

--

wpt-commits: 1b4fbe0c7049d89fd805dc157f68d82616f5d203, efc79eef01ffa65273ff01736d989ff5fa79f18d, 84af6c875d378944b39d895acdcfc170736b2d3d
wpt-pr: 17095

UltraBlame original commit: 696cf52f476f2f644137d19d9b9c4ec4dc400731
@whatwg whatwg deleted a comment Jul 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Define serialization and deserialization steps for DOMException
5 participants