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

Move ARIA attribute reflection tests from dom/nodes/ to html/dom/ #34299

Merged
merged 1 commit into from
Jun 3, 2022

Conversation

mrego
Copy link
Member

@mrego mrego commented Jun 3, 2022

As @annevk suggested in #34194 (comment), this is moving the tests out of the DOM spec folder.

The proposed spec text is being written in HTML spec: whatwg/html#7934

So HTML spec folder looks like a better place.

@mrego mrego merged commit ee566de into web-platform-tests:master Jun 3, 2022
@mrego mrego deleted the move-aria-reflection-tests branch June 3, 2022 12:59
aarongable pushed a commit to chromium/chromium that referenced this pull request Oct 4, 2022
The test was moved to the new folder in
web-platform-tests/wpt#34299

And was renamed in
web-platform-tests/wpt#34872

Change-Id: I64a9e23f0628929ff4b03e8430e96dc56666b932
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3933015
Reviewed-by: Frédéric Wang <[email protected]>
Commit-Queue: Manuel Rego <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1054655}
mjfroman pushed a commit to mjfroman/moz-libwebrtc-third-party that referenced this pull request Oct 14, 2022
The test was moved to the new folder in
web-platform-tests/wpt#34299

And was renamed in
web-platform-tests/wpt#34872

Change-Id: I64a9e23f0628929ff4b03e8430e96dc56666b932
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3933015
Reviewed-by: Frédéric Wang <[email protected]>
Commit-Queue: Manuel Rego <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1054655}
NOKEYCHECK=True
GitOrigin-RevId: 1a8eab305c730f004cdf9553059080d3a82a4344
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants