1451790 - Set Timer Resolution to 1ms with Jitter

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Tom Ritter [:tjr]]

Per discussion today, our plan is to go with 1ms with Jitter for 60/61.

Comment 1

[Tom Ritter [:tjr]]

Attachment: Bug 1451790 Set Time Precision to 1ms with Jitter

Review commit: https://reviewboard.mozilla.org/r/234120/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/234120/

Comment 2

[Andrea Marchesini [:baku]]

Comment on attachment 8965380 [details]
Bug 1451790 Set Time Precision to 1ms with Jitter

https://reviewboard.mozilla.org/r/234120/#review240588

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/eebb8d4e91d1
Set Time Precision to 1ms with Jitter r=baku

Comment 4

[Tom Ritter [:tjr]]

Attachment: 1451790-beta.patch

Approval Request Comment
[Feature/Bug causing the regression]: We can reduce the time precision from 2ms to 1ms on release.

[User impact if declined]: We'll have Release set to 2ms time precision and we can make it lower.

[Is this code covered by automated tests?]: Yes

[Has the fix been verified in Nightly?]: Yes

[Needs manual test from QE? If yes, steps to reproduce]: No

[List of other uplifts needed for the feature/fix]: None

[Is the change risky?]: No

[Why is the change risky/not risky?]: Just changing some constants

[String changes made/needed]: None

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/eebb8d4e91d1

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8966304 [details] [diff] [review]
1451790-beta.patch

Important mitigation needed for ESR60. Approved for 60.0b12.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/ab7ad29703d5

Comment 8

[Andrei Vaida [:avaida]]

Setting qe-verify- based on Tom's assessment on manual testing needs (Comment 4) and the fact that this fix has automated tests.