1854996 - Implement Permissions API extension for geckodriver

Status: RESOLVED FIXED

(Testing :: geckodriver, task, P3)

Description

[Kagami Rosylight [:saschanaz] (they/them)]

We do have pseudo support for that: https://searchfox.org/mozilla-central/rev/699a7704521354cac1e6a0679029c89ca899e25c/remote/marionette/permissions.sys.mjs#47-50

But that still requires setting prefs manually and thus causes lower wpt.fyi score.

It seems bug 1835920 added some code not depending on any prefs, but that code still doesn't apply for all permissions.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Please note that we also have bug 1524074 but that is for Marionette only.

Comment 2

[James Graham [:jgraham]]

We don't actually need this in geckodriver for wpt tests (except wdspec). So this is a valid bug, but it's bug 1524074 that's on the critical path for most wpt tests.

Comment 3

[Alexandra Borovova [:Sasha]]

Attachment: Bug 1854996 - Implement Permissions API extension for geckodriver.

Comment 4

[Alexandra Borovova [:Sasha]]

Attachment: Bug 1854996 - Remove Chrome specific test for permission command.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/99895876ea6f
Implement Permissions API extension for geckodriver. r=webdriver-reviewers,whimboo
https://hg.mozilla.org/integration/autoland/rev/f80eb710fa70
Remove Chrome specific test for permission command. r=webdriver-reviewers,whimboo

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/46645 for changes under testing/web-platform/tests

Comment 7

[Aron Cseh]

https://hg.mozilla.org/mozilla-central/rev/99895876ea6f
https://hg.mozilla.org/mozilla-central/rev/f80eb710fa70

Comment 8

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot