Closed Bug 1789484 Opened 2 years ago Closed 1 year ago

Return navigation id in browsingContext navigation events

Categories

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

Product:
Remote Protocol
Component:
WebDriver BiDi
Type:
task
Points:
3

Tracking

(firefox117 fixed)

Status:
RESOLVED FIXED
Milestone:
117 Branch
Tracking Flags:
Tracking Status
firefox117 --- fixed

People

(Reporter: whimboo, Assigned: jdescottes)

References

(Blocks 1 open bug)

Details

(Whiteboard: [webdriver:m8], [wptsync upstream] [webdriver:relnote])

Attachments

(2 files)

Bug 1789484 - [wdspec] Add tests for navigation id in browsingContext event tests
48 bytes, text/x-phabricator-request
Details | Review
Bug 1789484 - [bidi] Add navigation id to existing browsingContext events
48 bytes, text/x-phabricator-request
Details | Review

The navigation events from the browsingContext module have to return a valid navigation id. Right now such an id is not available yet.

We most likely have to wait until network events are available, and that we probably could create a unique navigation id based on their payload data.

Priority: -- → P3
Whiteboard: [webdriver:backlog]

Find potential blockers

Flags: needinfo?(jdescottes)

Hi Julian, could you please have a look here if that missing feature is something we could potentially work on in M7? If not it would be good to know which blockers exactly we have. I would like to discuss that in the next triage meeting. Thanks.

Whiteboard: [webdriver:backlog] → [webdriver:backlog][webdriver:triage]

As discussed in the meeting yesterday we will include this bug into M7 and we will try to get it working with the known workarounds. That will give us the necessary information for a potential platform enhancement later.

Flags: needinfo?(jdescottes)
Whiteboard: [webdriver:backlog][webdriver:triage] → [webdriver:backlog]
Points: --- → 5
Priority: P3 → P1
Whiteboard: [webdriver:backlog] → [webdriver:m7]
Priority: P1 → P2
Whiteboard: [webdriver:m7] → [webdriver:m8]
Points: 5 → 3
Depends on: 1841010
Type: enhancement → task
Whiteboard: [webdriver:m8] → [webdriver:m8:blocked]
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Attachment #9344237 - Attachment description: Bug 1789484 - [wdspec] Add tests for navigation id in browsingContext events → Bug 1789484 - [wdspec] Add tests for navigation id in browsingContext event tests
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/def8870bd849 [bidi] Add navigation id to existing browsingContext events r=webdriver-reviewers,jgraham https://hg.mozilla.org/integration/autoland/rev/59602a33332c [wdspec] Add tests for navigation id in browsingContext event tests r=webdriver-reviewers,jgraham
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/41124 for changes under testing/web-platform/tests
Whiteboard: [webdriver:m8:blocked] → [webdriver:m8:blocked], [wptsync upstream]

https://hg.mozilla.org/mozilla-central/rev/def8870bd849
https://hg.mozilla.org/mozilla-central/rev/59602a33332c

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox117: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
Whiteboard: [webdriver:m8:blocked], [wptsync upstream] → [webdriver:m8], [wptsync upstream]
Upstream PR merged by moz-wptsync-bot
Whiteboard: [webdriver:m8], [wptsync upstream] → [webdriver:m8], [wptsync upstream] [webdriver:relnote]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: