Skip to content

Consider using DOMHighResTimestamp instead of DOMTimeStamp #103

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
yoavweiss opened this issue Aug 31, 2021 · 5 comments · Fixed by #104
Closed

Consider using DOMHighResTimestamp instead of DOMTimeStamp #103

yoavweiss opened this issue Aug 31, 2021 · 5 comments · Fixed by #104

Comments

@yoavweiss
Copy link

We're considering our options RE DOMTimeStamp, and wondering what it's used for. It seems less well-defined than DOMHighResTimestamp. Would y'all consider switching over?

See whatwg/webidl#2 for discussion.

@marcoscaceres
Copy link
Member

marcoscaceres commented Aug 31, 2021

We probably can’t switch because of web compat. It’s used to determine the age of the acquired position.

But we could rename it LegacyGeolocationTimeStamp as it’s just a typedef and define it as used by the API. That could finally deprecating it from the platform.

@yoavweiss
Copy link
Author

Geolocation is not the only user today. Agree that renaming it may prevent new users though.

@marcoscaceres
Copy link
Member

Ah, my bad. Those WICG ones should be easy to update at least.

@npm1
Copy link

npm1 commented Oct 12, 2021

Compat aside, looks like DOMTimeStamp (or now EpochTimeStamp) introduce high resolution timers (there's not clamping whatsoever), so for privacy/security they should probably move?

@npm1
Copy link

npm1 commented Oct 12, 2021

Nevermind, Yoav mentioned this is an int so the granularity is not sub-ms.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants