Skip to content
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

Decreased DOMHighResTimeStamp resolution #55

Closed
wants to merge 1 commit into from
Closed

Conversation

plehegar
Copy link
Member

@plehegar plehegar commented Jan 5, 2018

Aligning with implementation expectation.


Preview | Diff

@plehegar plehegar added the security-tracker Group bringing to attention of security, or tracked by the security Group but not needing response. label Jan 5, 2018
@plehegar plehegar added this to the Level 2 milestone Jan 5, 2018
@igrigorik
Copy link
Member

@plehegar thanks for drafting this. Based on the discussion on the WG call today, the consensus was that we do not yet know what the exact recommendation here should be; each browser is doing something different: it looks like the current floor is 20us (Mozilla), but Edge is also adding jitter, Chrome is raising this to 100us for M64 and there are active investigations for M65+, and it's unclear what Webkit policy will be (they're already using clamping to 100us).

As a result, I don't think we know what the normative text should be here yet. Instead, perhaps we can add a warning in place pointing to this WIP discussion?

/cc @toddreifsteck

@plehegar
Copy link
Member Author

plehegar commented Jan 8, 2018

It might be better to open an issue in that case for the discussion, since this PR might become stale overtime. If you agree, I'll open one.

@igrigorik
Copy link
Member

Sounds good. Are you suggesting we also close this PR without further action? Or, are you suggesting adding a note and linking to an issue?

@igrigorik
Copy link
Member

From: https://webkit.org/blog/8048/what-spectre-and-meltdown-mean-for-webkit/

Timer precision from performance.now and other sources is reduced to 1ms (r226495).
We have disabled SharedArrayBuffer, since it can be used to create a high-resolution timer (r226386).

@plehegar
Copy link
Member Author

plehegar commented Jan 9, 2018

I opened #56 with all of the relevant links. Closing this PR since we need to get a better understanding first. I think a note in the meantime with a link to the issue is the best approach for now.

@plehegar plehegar closed this Jan 9, 2018
@plehegar plehegar deleted the plh/resolution branch January 11, 2018 13:25
@blurbusters
Copy link

blurbusters commented Feb 24, 2018

Crossposting here because someone at W3C gave me this link.

I have serious grave concerns about permanent precision reductions.

FireFox's 2ms precision now makes WebVR useless.

And a lot of 3D game programming techniques becomes impractical.

For a very good legitimate reason, I may have to begin automatically prompting users step by step instructions on overriding the precision reductions via advanced browser flags/config. This reduces security, since I essentially have to tell users to reduce the security globally just for one site - I do not like this approach.

Better remedy is a prompt/permissions based API like fullscreen, camera, VR, etc.

Please urgently read:
https://bugzilla.mozilla.org/show_bug.cgi?id=1427918#c9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement security-tracker Group bringing to attention of security, or tracked by the security Group but not needing response.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants