Skip to content

Update testdriver's set_permission methods to accept optional embeddedOrigin#55452

Merged
jcscottiii merged 7 commits intoweb-platform-tests:masterfrom
jalonthomas:update_testdriver
Oct 20, 2025
Merged

Update testdriver's set_permission methods to accept optional embeddedOrigin#55452
jcscottiii merged 7 commits intoweb-platform-tests:masterfrom
jalonthomas:update_testdriver

Conversation

@jalonthomas
Copy link
Copy Markdown
Contributor

@jalonthomas jalonthomas commented Oct 15, 2025

Align with addition: w3c/permissions#469
WPT: #55137
chromium-bidi change: GoogleChromeLabs/chromium-bidi#3799

@jalonthomas jalonthomas marked this pull request as ready for review October 16, 2025 14:48
@jalonthomas jalonthomas requested a review from a team as a code owner October 16, 2025 14:48
@wpt-pr-bot wpt-pr-bot added infra testdriver.js wptrunner The automated test runner, commonly called through ./wpt run labels Oct 16, 2025
@jalonthomas jalonthomas marked this pull request as draft October 16, 2025 14:49
@jalonthomas jalonthomas marked this pull request as ready for review October 20, 2025 13:28
@jcscottiii
Copy link
Copy Markdown
Contributor

jcscottiii commented Oct 20, 2025

Synced with @jalonthomas offline about the failing infrastructure test (/infrastructure/reftest/testdriver-iframe.sub.html). He's unable to repro this locally. Likely unrelated. Will force merge after one typical re-run.

@jcscottiii jcscottiii closed this Oct 20, 2025
@jcscottiii jcscottiii reopened this Oct 20, 2025
@jcscottiii
Copy link
Copy Markdown
Contributor

Force merging.

@jcscottiii jcscottiii merged commit c182610 into web-platform-tests:master Oct 20, 2025
43 of 45 checks passed
mertcanaltin pushed a commit to mertcanaltin/wpt that referenced this pull request Oct 26, 2025
…edOrigin` (web-platform-tests#55452)

* Add embedded_origin to testdriver

* Add param to WebDriverBidiPermissionsProtocolPart

* Finish updating to embeddedOrigin

* Add param types and return types

* Return awaitable instead of None

* Use Any return type
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra testdriver.js wptrunner The automated test runner, commonly called through ./wpt run

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants