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
Delete helper scripts in imported/w3c/web-platform-tests/file-system-access/resources #8846
Conversation
EWS run on current version of this PR (hash 7155d91) |
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.
These files seem to exist upstream: https://github.com/web-platform-tests/wpt/tree/master/file-system-access/resources , are we planning to remove them upstream as well?
Yes, they are unused since web-platform-tests/wpt@5798ab0. |
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.
Looks good, but please export the changes to WPT and have the files removed upstream as well.
7155d91
to
ec0e959
Compare
β¦access/resources https://bugs.webkit.org/show_bug.cgi?id=250855 rdar://104437010 Reviewed by Tim Nguyen. The tests that use these scripts were moved to fs/, and these scripts were copied to fs/resources/. * LayoutTests/imported/w3c/web-platform-tests/file-system-access/resources/message-target-dedicated-worker.js: Removed. * LayoutTests/imported/w3c/web-platform-tests/file-system-access/resources/message-target-service-worker.js: Removed. * LayoutTests/imported/w3c/web-platform-tests/file-system-access/resources/message-target-shared-worker.js: Removed. * LayoutTests/imported/w3c/web-platform-tests/file-system-access/resources/message-target.html: Removed. * LayoutTests/imported/w3c/web-platform-tests/file-system-access/resources/message-target.js: Removed. * LayoutTests/imported/w3c/web-platform-tests/file-system-access/resources/messaging-blob-helpers.js: Removed. * LayoutTests/imported/w3c/web-platform-tests/file-system-access/resources/messaging-helpers.js: Removed. * LayoutTests/imported/w3c/web-platform-tests/file-system-access/resources/messaging-serialize-helpers.js: Removed. Canonical link: https://commits.webkit.org/259120@main
ec0e959
to
fe46c0d
Compare
Committed 259120@main (fe46c0d): https://commits.webkit.org/259120@main Reviewed commits have been landed. Closing PR #8846 and removing active labels. |
wpt pull request: web-platform-tests/wpt#38073 |
fe46c0d
7155d91
π π§ͺ winπ wincairoπ§ͺ ios-wk2π§ͺ mac-AS-debug-wk2