Closed Bug 1713439 Opened 4 months ago Closed 2 months ago

MessageHandler: Initial MessageHandler support for content windows

Categories

(Remote Protocol :: WebDriver BiDi, task, P2)

task
Points:
13

Tracking

(firefox92 fixed)

RESOLVED FIXED
92 Branch
Tracking Status
firefox92 --- fixed

People

(Reporter: jdescottes, Assigned: jdescottes)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [bidi-m1-mvp])

Attachments

(7 files, 3 obsolete files)

Land a basic CommandsHandler support, which can only target content window global contexts.

This should be similar to what is done in the prototype patch https://phabricator.services.mozilla.com/D115473

Blocks: 1713440
Blocks: 1713441

Note that the basic architecture should already satisfy the main requirements for CommandsHandler, meaning:

  • spawn modules in any relevant layer (parent process or content process main thread here)
  • support multiple sessions
  • route commands from the parent process to content process main thread context
  • intercept commands in the parent process targeting a content window global context
Whiteboard: [bidi-m1-mvp]
Points: --- → 8
Priority: -- → P2
Points: 8 → 13
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Attachment #9231607 - Attachment description: Bug 1713439 - [remote] EXAMPLE: Add fake mochitest to exercize message-handler API → Bug 1713439 - [remote] TEMPORARY: Add mochitests to exercize message-handler API
No longer blocks: 1713784
Depends on: 1713784
Attached image MessageHandler_scenario_2.svg (obsolete) —
Attached image MessageHandler_scenario_3.svg (obsolete) —
Attachment #9231961 - Attachment is obsolete: true
Attachment #9231969 - Attachment is obsolete: true
Attachment #9233095 - Attachment is obsolete: true
Attachment #9233095 - Attachment is obsolete: false
Attachment #9231971 - Attachment is obsolete: true

Fixed the SVG diagrams.

Summary: CommandsHandler: Initial CommandsHandler support for content windows → MessageHandler: Initial MessageHandler support for content windows
Attachment #9231607 - Attachment description: Bug 1713439 - [remote] TEMPORARY: Add mochitests to exercize message-handler API → Bug 1713439 - [remote] Add mochitests to exercize message-handler API
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6f355f7623d5
[remote] Add basic MessageHandler support for window global contexts r=webdriver-reviewers,whimboo,jgraham
https://hg.mozilla.org/integration/autoland/rev/6bc2c4e3190f
[remote] Add logs to initial MessageHandler classes r=webdriver-reviewers,whimboo
https://hg.mozilla.org/integration/autoland/rev/26d7b5c2a6ae
[remote] Use a MessageHandler to handle packets received by the WebDriverBiDiConnection r=webdriver-reviewers,whimboo
https://hg.mozilla.org/integration/autoland/rev/99c88470ce60
[remote] Add mochitests to exercize message-handler API r=webdriver-reviewers,whimboo
You need to log in before you can comment on or make changes to this bug.