-
Notifications
You must be signed in to change notification settings - Fork 26
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
feat(script): implement user activation #1105
Conversation
5ad8455
to
d88b02e
Compare
🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use -- conventional-commit-lint bot |
d88b02e
to
f914661
Compare
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.
Could you also simplify these two tests?
chromium-bidi/tests/test_browsing_context.py
Line 1144 in 4b1945f
async def test_browsingContext_create_withUserGesture_eventsEmitted( chromium-bidi/tests/test_preload_script.py
Line 675 in 4b1945f
async def test_preloadScript_add_withUserGesture_blankTargetLink(
See the XXX
comment there related to w3c/webdriver-bidi#359.
f914661
to
7326af3
Compare
542bc22
to
2e4dc7f
Compare
Please watch out for an upcoming merge conflict: #1110 Tests are now organized per domain. |
2e4dc7f
to
44ca47b
Compare
44ca47b
to
79967bc
Compare
🤖 I have created a release *beep* *boop* --- ## [0.4.20](chromium-bidi-v0.4.19...chromium-bidi-v0.4.20) (2023-08-01) ### Features * implement `browser.close` ([#1116](#1116)) ([a715559](a715559)) * **script:** implement user activation ([#1105](#1105)) ([2408d7f](2408d7f)) ### Bug Fixes * don't publish `.tsbuildinfo` ([#1106](#1106)) ([4b1945f](4b1945f)) * use Result passing to prevent Unhandled promise rejections ([#1112](#1112)) ([e0dc19f](e0dc19f)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
The rest of tests will be covered by WPT.