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

[Gecko Bug 1522790] Use marionette new window command to open wptrunner windows #45735

Merged
merged 4 commits into from
Sep 23, 2024

Conversation

moz-wptsync-bot
Copy link
Collaborator

Differential Revision: https://phabricator.services.mozilla.com/D23972

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1522790
gecko-commit: ed9818780499bbba8c5b606bd2632985ee3e5991
gecko-reviewers: jgraham

@wpt-pr-bot wpt-pr-bot added css-scoping html infra wptrunner The automated test runner, commonly called through ./wpt run labels Apr 16, 2024
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Firefox project.

@whimboo
Copy link
Contributor

whimboo commented Sep 23, 2024

All the tests are passing now but it looks like that something is wrong with the sync task not ending correctly.

WE most likely need an admin merge here. CC @web-platform-tests/admins

jgraham and others added 3 commits September 23, 2024 14:49
Differential Revision: https://phabricator.services.mozilla.com/D23972

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1522790
gecko-commit: ed9818780499bbba8c5b606bd2632985ee3e5991
gecko-reviewers: jgraham
Depends on D23972

Differential Revision: https://phabricator.services.mozilla.com/D204234

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1522790
gecko-commit: d099e86e9a3f35890fbec91fef48fc0115f03480
gecko-reviewers: whimboo
…arguments.

Depends on D206622

Differential Revision: https://phabricator.services.mozilla.com/D206623

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1522790
gecko-commit: 91d70d27459b254b2b9d8cf4b610c09e593ef369
gecko-reviewers: jgraham
@moz-wptsync-bot moz-wptsync-bot merged commit 1700d34 into master Sep 23, 2024
42 checks passed
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1522790-2 branch September 23, 2024 13:41
jonathan-j-lee added a commit to jonathan-j-lee/wpt that referenced this pull request Oct 14, 2024
Similar to web-platform-tests#48106, permission changes may leak between tests that use
the same WebDriver session (i.e., browser process). For now, add a
Chromium-specific `Browser.resetPermissions` call to reset permission
defaults.

To consolidate calls that reset state, refactor the WebDriver executor's
testharness window management to align with web-platform-tests#45735.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
html infra mozilla:gecko-sync wptrunner The automated test runner, commonly called through ./wpt run
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants