Closed Bug 1702335 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 28320 - Coarsen `performance.timeOrigin()`

Categories

(Core :: DOM: Core & HTML, task, P4)

Product:
Core
Component:
DOM: Core & HTML
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
89 Branch
Tracking Flags:
Tracking Status
firefox89 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 28320 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/28320
Details from upstream follow.

b'Yoav Weiss <yoavweiss@chromium.org>' wrote:

Coarsen performance.timeOrigin()

performance.timeOrigin() is not currently coarsened, which is not
ideal.
This CL fixes that. It also moves the clamping to use ints (in
microseconds), rather than doubles (in seconds), because
https://bit.ly/doubles-are-bad

I2S: https://groups.google.com/a/chromium.org/g/blink-dev/c/k6M3HJiqmkE/m/zgaVyKfnAwAJ

Bug: 1180178
Change-Id: Id23637da59b01383a450a9e75ff4fb3f2825c46b
Reviewed-on: https://chromium-review.googlesource.com/2780084
WPT-Export-Revision: d5605b98ebc23c663163cc900eac8f5fed25c75f

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core

CI Results

Ran 15 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 2 tests and 1 subtests

Status Summary

Firefox

OK : 2
PASS: 1[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview] 2[Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, Gecko-windows7-32-debug, Gecko-windows7-32-opt, GitHub]
FAIL: 1

Chrome

OK : 2
PASS: 2

Safari

OK : 2
FAIL: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/hr-time/clamped-time-origin-isolated.https.html
timeOrigins are clamped.: FAIL [Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview], PASS [Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, Gecko-windows7-32-debug, Gecko-windows7-32-opt, GitHub] (Chrome: PASS, Safari: FAIL)

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6e7d4a1c29ac [wpt PR 28320] - Coarsen `performance.timeOrigin()`, a=testonly https://hg.mozilla.org/integration/autoland/rev/ff6c3b454b5a [wpt PR 28320] - Update wpt metadata, a=testonly
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f6406350cadd [wpt PR 28320] - Coarsen `performance.timeOrigin()`, a=testonly https://hg.mozilla.org/integration/autoland/rev/638cc979a5db [wpt PR 28320] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/f6406350cadd
https://hg.mozilla.org/mozilla-central/rev/638cc979a5db

Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox89: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in before you can comment on or make changes to this bug.