1920959 - Remove action processing in content process, including the preference "remote.events.async.enabled"

Status: RESOLVED FIXED

(Remote Protocol :: Agent, task, P3)

Description

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

Once all actions are getting processed and dispatched via the parent process we can remove the fallback code for handling actions in the content process. This includes as well the usage of the remote.events.async.enabled preference.

Comment 1

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

With the work on bug 1848957, we’re introducing additional code paths for processing wheel events in the parent process, which makes the code harder to maintain and follow. Since action processing in the parent process has been shipped since Firefox 135 and we’ve only seen a single regression reported, I believe it’s reasonable to remove the fallback code along with the associated preference.

Additionally, we could remove the WPT and wdspec variant jobs, as they are tier 3 and currently not being monitored—while still consuming CI resources.

Comment 2

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

Attachment: Bug 1920959 - [marionette] Remove fallback for action processing in the content process.

Comment 3

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

Attachment: Bug 1920959 - [webdriver-bidi] Remove fallback for action processing in the content process.

Comment 4

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

Attachment: Bug 1920959 - [remote] Remove preference "remote.events.async.enabled".

Comment 5

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

Attachment: Bug 1920959 - [remote] Remove async-events-dispatching jobs.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/75aa3c6fbe54
[marionette] Remove fallback for action processing in the content process. r=webdriver-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/d636677ad717
[webdriver-bidi] Remove fallback for action processing in the content process. r=webdriver-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/13905de40583
[remote] Remove preference "remote.events.async.enabled". r=webdriver-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/4dfc09dfe7a3
[remote] Remove async-events-dispatching jobs. r=webdriver-reviewers,jdescottes

Comment 7

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

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

Comment 8

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

Attachment: Bug 1920959 - [marionette] Fix linter failure in driver.sys.mjs.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/63550d3fa99c
[marionette] Fix linter failure in driver.sys.mjs. CLOSED TREE

Comment 10

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/75aa3c6fbe54
https://hg.mozilla.org/mozilla-central/rev/d636677ad717
https://hg.mozilla.org/mozilla-central/rev/13905de40583
https://hg.mozilla.org/mozilla-central/rev/4dfc09dfe7a3
https://hg.mozilla.org/mozilla-central/rev/63550d3fa99c

Comment 11

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

(In reply to Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org) from comment #7)

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

The PR got merged.