Closed Bug 1694143 Opened 7 months ago Closed 12 days ago

Implement basic support for log.entryAdded

Categories

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

task
Points:
8

Tracking

(firefox94 fixed)

RESOLVED FIXED
94 Branch
Tracking Status
firefox94 --- fixed

People

(Reporter: whimboo, Assigned: jdescottes)

References

(Blocks 4 open bugs)

Details

(Whiteboard: [bidi-m1-mvp], [wptsync upstream])

Attachments

(4 files)

No description provided.
Points: --- → 8
Priority: -- → P2
Blocks: 1719287

For the demo at the end of M1 we only need the basic support for this events means handling of just string messages. Anything else will be done at a later point and tracked via the meta bug 1719287.

Summary: Implement log.entryAdded → Implement basic support for log.entryAdded
Depends on: 1694144
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED

When using the BiDiSession class we need the event loop to live at
least as long as a specific connection, but not necessarily as long as
the BiDISession object itself. Therefore we move the loop parameter
into the start() method, to associate a loop with a specific connection.

In addition, in the tests we make the event loop a global so that it
lives as long as a session; the fixture is updated to reuse the
existing loop where possible. This has the tradeoff that we're more
likely to share a loop between tests, so there is some additional
hidden shared state.

Attachment #9239917 - Attachment description: Bug 1694143 - Ensure that asyncio event loop outlives webdriver connection, → Bug 1694143 - [remote] Ensure that asyncio event loop outlives webdriver connection
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/002f6e22802f
[remote] Add log module and support log.entryAdded event r=webdriver-reviewers,whimboo
https://hg.mozilla.org/integration/autoland/rev/798401a59fee
[remote] Return remove_listener function from wdspec bidi client BiDiSession::add_event_listener r=webdriver-reviewers,whimboo
https://hg.mozilla.org/integration/autoland/rev/908c3e478858
[remote] Ensure that asyncio event loop outlives webdriver connection r=webdriver-reviewers,whimboo
https://hg.mozilla.org/integration/autoland/rev/9e1434237651
[remote] Add wdspec tests for log.entryAdded r=webdriver-reviewers,whimboo
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/30486 for changes under testing/web-platform/tests
Whiteboard: [bidi-m1-mvp] → [bidi-m1-mvp], [wptsync upstream]
Upstream PR merged by jgraham
You need to log in before you can comment on or make changes to this bug.