"browsingContext.reload" command has to return a "NavigateResult"
Categories
(Remote Protocol :: WebDriver BiDi, enhancement, P1)
Tracking
(firefox119 fixed)
Tracking | Status | |
---|---|---|
firefox119 | --- | fixed |
People
(Reporter: whimboo, Assigned: whimboo)
References
(Blocks 1 open bug, )
Details
(Whiteboard: [webdriver:m8], [wptsync upstream])
Attachments
(2 files)
Bug 1851583 - [webdriver-bidi] Update "browsingContext.reload" command to return a "NavigateResult".
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review |
As per the BiDi specification there is no NavigationResult
returned at the moment, which is needed by clients to actually listen for certain events. I filed https://github.com/w3c/webdriver-bidi/issues/527 to get this fixed.
Once the specification has been updated we have to return a NavigationResult
response.
Assignee | ||
Comment 1•8 months ago
|
||
The WebDriver BiDi issue has been fixed. So we can get this fixed once bug 1830859 landed.
Assignee | ||
Comment 2•8 months ago
|
||
Actually the BiDi issue got reopened and there is a discussion about the navigation id. As such lets wait until we have a clear path forward.
Assignee | ||
Comment 3•8 months ago
|
||
When working on this bug it would be good to actually address the review comment from https://phabricator.services.mozilla.com/D187445#6204269 which raced with the landing request and couldn't applied anymore.
Assignee | ||
Comment 4•8 months ago
|
||
For the upstream issue we decided to file a follow-up issue for the special case, and as such closed it. That means we can get started to implement the NavigateResult
. I can have a look at this later today.
Assignee | ||
Comment 5•8 months ago
|
||
Updated•8 months ago
|
Assignee | ||
Comment 6•8 months ago
|
||
Depends on D187754
Assignee | ||
Updated•8 months ago
|
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/00e0560c4f72 [webdriver-bidi] Update "browsingContext.reload" command to return a "NavigateResult". r=webdriver-reviewers,jdescottes https://hg.mozilla.org/integration/autoland/rev/371f21acf806 [wdspec] Update tests for browsingContext.reload to expect a "NavigateResult". r=webdriver-reviewers,jdescottes
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/41911 for changes under testing/web-platform/tests
Upstream PR merged by thiagowfx
Comment 10•8 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/00e0560c4f72
https://hg.mozilla.org/mozilla-central/rev/371f21acf806
Description
•