Closed Bug 1916685 Opened 3 months ago Closed 3 months ago

Fetch timings should be in milliseconds, not microseconds

Categories

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

defect
Points:
3

Tracking

(firefox132 fixed)

RESOLVED FIXED
132 Branch
Tracking Status
firefox132 --- fixed

People

(Reporter: jdescottes, Assigned: jdescottes)

References

(Blocks 4 open bugs)

Details

(Whiteboard: [webdriver:m12][webdriver:relnote][wptsync upstream])

Attachments

(2 files)

Per spec, the network event timestamps should be in milliseconds, however we return a value in microseconds at the moment.

https://www.w3.org/TR/hr-time-3/#dom-domhighrestimestamp

All timings from platform are in microseconds, and we didn't think about converting them.

Duplicate of this bug: 1916651

Platform timestamps are in microseconds, while we expect network event timings in milliseconds

Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Severity: -- → S3
Points: --- → 3
Priority: -- → P2
Whiteboard: [webdriver:m12]
Attachment #9422581 - Attachment description: Bug 1916685 - [wdspec] Update network bidi tests to assert that timings are within a reasonable range → Bug 1916685 - [wdspec] Update several network bidi tests to assert timings
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d9a88df573cb [bidi] Convert all network event timings to milliseconds r=webdriver-reviewers,whimboo https://hg.mozilla.org/integration/autoland/rev/e1aade50c0eb [wdspec] Update several network bidi tests to assert timings r=webdriver-reviewers,whimboo
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/48277 for changes under testing/web-platform/tests
Whiteboard: [webdriver:m12] → [webdriver:m12], [wptsync upstream]
Upstream PR merged by moz-wptsync-bot

The BiDi har exporter was updated to handle both timings in milliseconds or microseconds in https://github.com/firefox-devtools/bidi-har-export/releases/tag/v0.0.15

Whiteboard: [webdriver:m12], [wptsync upstream] → [webdriver:m12][webdriver:relnote][wptsync upstream]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: