Implement full support for "session.subscribe" and "session.unsubscribe" commands
Categories
(Remote Protocol :: WebDriver BiDi, task, P1)
Tracking
(firefox109 fixed)
| Tracking | Status | |
|---|---|---|
| firefox109 | --- | fixed |
People
(Reporter: whimboo, Assigned: Sasha)
References
(Blocks 4 open bugs, )
Details
(Whiteboard: [webdriver:m5], [wptsync upstream], [webdriver:relnote] )
Attachments
(3 files, 1 obsolete file)
With bug 1694144 basic support for the session.subscribe command will be added so that it just works for log.entryAdded. To be fully BiDi compatible more work will be necessary.
We should follow the following part of the specification, and if needed file dependency bugs if blockers come up:
https://w3c.github.io/webdriver-bidi/#command-session-subscribe
|
Reporter | |
Comment 1•3 years ago
|
||
|
||
Updated•3 years ago
|
|
|
Reporter | |
Comment 2•3 years ago
|
||
Only added a WIP that contains code that we do not yet want to see in the session module for bug 1694144.
|
|
||
Updated•3 years ago
|
|
|
Reporter | |
Updated•3 years ago
|
|
|
Reporter | |
Updated•3 years ago
|
|
|
Reporter | |
Updated•3 years ago
|
|
|
Reporter | |
Updated•3 years ago
|
|
|
Reporter | |
Updated•3 years ago
|
|
|
Reporter | |
Comment 3•3 years ago
|
||
Not actually working on this bug right now given that it is blocked on other bugs that need to be fixed first.
|
|
Reporter | |
Updated•3 years ago
|
|
||
Updated•2 years ago
|
|
|
||
Comment 4•2 years ago
|
||
In Bug 1694145 we will land a basic implementation for session.unsubscribe which will match our current implementation for session.subscribe.
After that we should implement features simultaneously for subscribe and unsubscribe, so this bug is repurposed to handle the complete implementation for both subscribe and unsubscribe.
|
|
||
Updated•2 years ago
|
|
|
||
Updated•2 years ago
|
|
|
Reporter | |
Updated•2 years ago
|
|
|
Reporter | |
Updated•2 years ago
|
|
|
Reporter | |
Comment 6•2 years ago
|
||
I'll have to split this bug into two (subscribe and unsubscribe). I'll do this later today.
|
|
Reporter | |
Comment 7•2 years ago
|
||
As discussed today in the triage meeting we are going to split-up this test into session.subscribe and session.unsubscribe. I'm going to file a new bug for the latter command in a minute.
|
Assignee | |
Updated•2 years ago
|
|
|
Assignee | |
Comment 8•2 years ago
|
||
|
|
Assignee | |
Comment 9•2 years ago
|
||
Depends on D161307
|
|
Assignee | |
Comment 10•2 years ago
|
||
Depends on D161308
|
|
||
Updated•1 year ago
|
|
|
||
Updated•1 year ago
|
|
|
||
Updated•1 year ago
|
|
|
Reporter | |
Updated•1 year ago
|
|
|
||
Updated•1 year ago
|
|
|
||
Updated•1 year ago
|
|
||
Comment 12•1 year ago
|
||
Pushed by aborovova@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fb432fd5b045 [bidi] Implement full support for session.subscribe and session.unsubscribe commands. r=webdriver-reviewers,jdescottes,whimboo https://hg.mozilla.org/integration/autoland/rev/6b8a1f1803f8 [wdpspec] Add bidi_subscribe fixture to automatically unsubscribe at the end of a test. r=webdriver-reviewers,jdescottes,whimboo https://hg.mozilla.org/integration/autoland/rev/bec825dafefb [wdpspec] Add tests for session.subscribe and session.unsubscribe. r=webdriver-reviewers,jdescottes,whimboo
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/37400 for changes under testing/web-platform/tests
|
||
Comment 14•1 year ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/fb432fd5b045
https://hg.mozilla.org/mozilla-central/rev/6b8a1f1803f8
https://hg.mozilla.org/mozilla-central/rev/bec825dafefb
Upstream PR was closed without merging
Upstream PR merged by jgraham
|
|
||
Updated•1 year ago
|
Description
•