Closed Bug 1451790 Opened 2 years ago Closed 2 years ago

Set Timer Resolution to 1ms with Jitter

Categories

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

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox60 + fixed
firefox61 + fixed

People

(Reporter: tjr, Assigned: tjr)

References

(Depends on 1 open bug)

Details

Attachments

(2 files)

Per discussion today, our plan is to go with 1ms with Jitter for 60/61.
Comment on attachment 8965380 [details]
Bug 1451790 Set Time Precision to 1ms with Jitter

https://reviewboard.mozilla.org/r/234120/#review240588
Attachment #8965380 - Flags: review?(amarchesini) → review+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/eebb8d4e91d1
Set Time Precision to 1ms with Jitter r=baku
Keywords: checkin-needed
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
Attachment #8966304 - Flags: approval-mozilla-beta?
https://hg.mozilla.org/mozilla-central/rev/eebb8d4e91d1
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Comment on attachment 8966304 [details] [diff] [review]
1451790-beta.patch

Important mitigation needed for ESR60. Approved for 60.0b12.
Attachment #8966304 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Setting qe-verify- based on Tom's assessment on manual testing needs (Comment 4) and the fact that this fix has automated tests.
Flags: qe-verify-
Component: DOM: Events → DOM
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.