Support the context parameter for browsingContext.create
Categories
(Remote Protocol :: WebDriver BiDi, task, P1)
Tracking
(firefox107 fixed)
| Tracking | Status | |
|---|---|---|
| firefox107 | --- | fixed |
People
(Reporter: jdescottes, Assigned: jdescottes)
References
(Blocks 2 open bugs, )
Details
(Whiteboard: [webdriver:m5], [wptsync upstream])
Attachments
(6 files)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review |
The context parameter is proposed at https://github.com/w3c/webdriver-bidi/pull/153
It allows to specify a reference context for browsingContext.create. When type "tab" is used the new context should share the same OS window as the reference context.
This bug is about adding the implementation for this parameter, but we should at least wait until the spec PR is close to be accepted to move forward.
|
Assignee | |
Comment 1•2 years ago
|
||
Depends on D143650
|
||
Updated•2 years ago
|
|
|
Assignee | |
Comment 2•2 years ago
|
||
Depends on D143651
|
|
Assignee | |
Comment 3•2 years ago
|
||
Depends on D143652
|
|
Assignee | |
Comment 4•2 years ago
|
||
Depends on D143653
|
||
Updated•2 years ago
|
|
|
||
Updated•2 years ago
|
|
|
Assignee | |
Updated•2 years ago
|
|
|
||
Comment 5•2 years ago
|
||
The PR got merged so this bug is no longer blocked. Julian, do you want to continue working on it?
|
|
Assignee | |
Comment 6•2 years ago
|
||
I will at least rebase and update the patches to match the PR, we can discuss if that should be in m5 then.
|
|
Assignee | |
Comment 7•2 years ago
|
||
The updated version of the browsingContext.create spec mentions that the new tab should be added right after the reference context when
provided. We need to extend TabManager.addTab in order to support this behavior:
|
|
Assignee | |
Updated•2 years ago
|
|
|
||
Updated•2 years ago
|
|
|
||
Updated•2 years ago
|
|
|
Assignee | |
Comment 8•2 years ago
|
||
Depends on D159121
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f497dc5418fb [remote] Add referenceTab parameter to TabManager addTab r=webdriver-reviewers,Sasha,whimboo https://hg.mozilla.org/integration/autoland/rev/bfcad162f26d [remote] Add getTabForBrowsingContext helper in TabManager r=webdriver-reviewers,whimboo https://hg.mozilla.org/integration/autoland/rev/d9ddc65fcb38 [webdriver-bidi] Add context parameter to browsingContext.create command r=webdriver-reviewers,Sasha,whimboo https://hg.mozilla.org/integration/autoland/rev/744c82b97d76 [webdriver] Support context parameter context for browsing context create r=webdriver-reviewers,Sasha,whimboo https://hg.mozilla.org/integration/autoland/rev/6f36f9508aa4 [wdspec] Update wdspec tests for the browsingContext.create context parameter r=webdriver-reviewers,whimboo https://hg.mozilla.org/integration/autoland/rev/9335bd19ed0d [webdriver-bidi] Add gecko specific webdriver test for browsingContext.create context argument r=webdriver-reviewers,whimboo
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/36466 for changes under testing/web-platform/tests
|
||
Comment 11•2 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/f497dc5418fb
https://hg.mozilla.org/mozilla-central/rev/bfcad162f26d
https://hg.mozilla.org/mozilla-central/rev/d9ddc65fcb38
https://hg.mozilla.org/mozilla-central/rev/744c82b97d76
https://hg.mozilla.org/mozilla-central/rev/6f36f9508aa4
https://hg.mozilla.org/mozilla-central/rev/9335bd19ed0d
Upstream PR merged by moz-wptsync-bot
Description
•