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
Origin isolation: refactor and deduplicate web platform tests #23964
Conversation
There was a problem hiding this 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.
Now that I've written a bunch of them, I'm confident in building some abstractions to deduplicate the testing code. This additionally adds jsdoc documentation for all of the exported functions in helpers.mjs. Bug: 1042415 Change-Id: I3e2220708f4d5afff5a815de876971db3f689826 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2229435 Reviewed-by: James MacLean <wjmaclean@chromium.org> Commit-Queue: Domenic Denicola <domenic@chromium.org> Cr-Commit-Position: refs/heads/master@{#777962}
2f2ec57
to
f3d4e8d
Compare
Fails chrome-dev stability check:
I'm unable to reproduce locally (via |
Now that I've written a bunch of them, I'm confident in building some
abstractions to deduplicate the testing code.
This additionally adds jsdoc documentation for all of the exported
functions in helpers.mjs.
Bug: 1042415
Change-Id: I3e2220708f4d5afff5a815de876971db3f689826
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2229435
Reviewed-by: James MacLean <wjmaclean@chromium.org>
Commit-Queue: Domenic Denicola <domenic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#777962}