Closed Bug 1851511 Opened 1 year ago Closed 1 year ago

Allow to send widget events by Remote Protocols from the content process

Categories

(Core :: DOM: Content Processes, enhancement)

enhancement

Tracking

()

RESOLVED WONTFIX

People

(Reporter: whimboo, Assigned: whimboo)

Details

Attachments

(1 obsolete file)

For bug 1848957 the event handling code in Marionette and the Remote Agent need to send widget events from the content process. This doesn't work right now because eg. in BrowserParent::RecvDispatchWheelEvent a check for Cu.isInAutomation prevents that and causes the browser to crash.

As discussed with Nika we should have a separate check for our Remote Protocols (Marionette and Remote Agent) beside the one for Automation, so that running tests with Marionette, and WebDriver tests in general will always work and that especially outside of our CI.

Remote protocols like Marionette and Remote Agent that are also
used for WebDriver tests need to be able to simulate events
as close as possible to the real user. As such widget events
will be used to allow user input emulation for both headful
and headless testing.

Hereby it is not enough to only check for Cu.isInAutomation
because WebDriver tests will be mostly run outside of our CI
systems, and both remote protocols cannot fulfill the requirements
for Cu.isInAutomation. Therefore a separate check is used.

Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Attachment #9351477 - Attachment description: WIP: Bug 1851511 - [dom] Allow to send widget events by Remote Protocols from the content process. → Bug 1851511 - [dom] Allow to send widget events by Remote Protocols from the content process.
Attachment #9351477 - Attachment is obsolete: true

After talking to Olli today we decided to not follow this proposal but instead get Marionette and Remote Agent to dispatch the events on the parent process. The meta bug for this work is bug 1851812.

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX
No longer blocks: 1848957
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: