Skip to content

Fix error in "get a known shadow root" algorithm #1726

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

Merged
merged 1 commit into from
Mar 21, 2023

Conversation

lutien
Copy link
Contributor

@lutien lutien commented Mar 15, 2023

Copy link
Contributor

@whimboo whimboo left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks Sasha!

@sadym-chromium mind having a quick look as well? Thanks.

@whimboo
Copy link
Contributor

whimboo commented Mar 21, 2023

Small bug fix so we probably do not have to wait for further reviewers. I'm going ahead and merge this PR. Thanks @lutien.

@whimboo whimboo merged commit 231c5fb into w3c:master Mar 21, 2023
github-actions bot added a commit that referenced this pull request Mar 21, 2023
SHA: 231c5fb
Reason: push, by whimboo

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@whimboo
Copy link
Contributor

whimboo commented Mar 21, 2023

The wdspec tests have already been updated and a test for that change can be found here.

github-actions bot added a commit to xjc90s/webdriver that referenced this pull request Mar 21, 2023
SHA: 231c5fb
Reason: push, by pull[bot]

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit to soloinovator/webdriver that referenced this pull request Mar 21, 2023
SHA: 231c5fb
Reason: push, by pull[bot]

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants