1790378 - Add timestamp to browsingContext navigation events

Status: RESOLVED FIXED

(Remote Protocol :: WebDriver BiDi, enhancement, P1)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

As of now all the browsingContext related navigation events do not have a timestamp set. To measure performance based on these events a timestamp should be added.

There is the following GitHub issue open to get this entry added:
https://github.com/w3c/webdriver-bidi/issues/271

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

The WebDriver BiDi specification PR has been merged. As such we can get the timestamp added now.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Attachment: Bug 1790378 - [webdriver-bidi] Added timestamp to navigation info.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Attachment: Bug 1790378 - [wdspec] Enhance navigation tests for "timestamp" in navigation info.

Depends on D166381

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Currently we only have browsingContext.domContentLoaded and browsingContext.load implemented.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8c175c957f35
[webdriver-bidi] Added timestamp to navigation info. r=webdriver-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/7c1c3dbacb6c
[wdspec] Enhance navigation tests for "timestamp" in navigation info. r=webdriver-reviewers,jdescottes

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Failed to create upstream wpt PR due to merge conflicts. This requires fixup from a wpt sync admin.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

(In reply to Web Platform Test Sync Bot (Matrix: #interop:mozilla.org) from comment #6)

Failed to create upstream wpt PR due to merge conflicts. This requires fixup
from a wpt sync admin.

Hi James, could you please check why the upstream PR could not be created? Thanks!

Comment 8

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/37834 for changes under testing/web-platform/tests

Comment 9

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/8c175c957f35
https://hg.mozilla.org/mozilla-central/rev/7c1c3dbacb6c

Comment 10

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot