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)
Remote Protocol
WebDriver BiDi
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.
Assignee | ||
Updated•3 months ago
|
Assignee | ||
Comment 2•3 months ago
|
||
Platform timestamps are in microseconds, while we expect network event timings in milliseconds
Updated•3 months ago
|
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•3 months ago
|
||
Depends on D221033
Assignee | ||
Updated•3 months ago
|
Severity: -- → S3
Points: --- → 3
Priority: -- → P2
Whiteboard: [webdriver:m12]
Updated•3 months ago
|
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
Comment 5•3 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d9a88df573cb
https://hg.mozilla.org/mozilla-central/rev/e1aade50c0eb
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
status-firefox132:
--- → fixed
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
Assignee | ||
Comment 8•3 months ago
|
||
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
Updated•2 months ago
|
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.
Description
•