Closed Bug 1429668 Opened 2 years ago Closed 2 years ago

Intermittent dom/animation/test/css-animations/test_animation-starttime.html | Sanity test to check round-tripping assigning to a new animation's startTime - Sanity test to check round-tripping assigning to a new animation's startTime:

Categories

(Core :: DOM: Animation, defect, P5)

defect

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox-esr52 --- unaffected
firefox57 --- unaffected
firefox58 --- fixed
firefox59 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: tjr)

Details

(Keywords: intermittent-failure, Whiteboard: [stockwell fixed:timing])

Attachments

(1 file)

Attachment #8941714 - Flags: review+
Comment on attachment 8941714 [details]
Bug 1429668 Turn off the reduceTimerPrecision pref for one more test

https://reviewboard.mozilla.org/r/211946/#review217752

::: dom/animation/test/css-animations/test_animation-starttime.html:10
(Diff revision 1)
>  <div id="log"></div>
>  <script>
>  'use strict';
>  setup({explicit_done: true});
>  SpecialPowers.pushPrefEnv(
> -  { "set": [["dom.animations-api.core.enabled", true]]},
> +  { "set": 

Nit: a trailing space.
Assignee: nobody → tom
Status: NEW → ASSIGNED
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/autoland/rev/bb887872b8e5
Turn off the reduceTimerPrecision pref for one more test r=hiro
Pushed by apavel@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/2cb19d6666db
Turn off the reduceTimerPrecision pref for one more test r=hiro a=test-fix
https://hg.mozilla.org/mozilla-central/rev/2cb19d6666db
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Recent failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=155603058&repo=mozilla-inbound&lineNumber=1956

[task 2018-01-11T12:02:30.940Z] 12:02:30     INFO -  177 INFO TEST-START | dom/animation/test/css-animations/test_animation-starttime.html
[task 2018-01-11T12:02:41.230Z] 12:02:41     INFO -  Buffered messages logged at 12:02:31
[task 2018-01-11T12:02:41.230Z] 12:02:41     INFO -  178 INFO TEST-PASS | dom/animation/test/css-animations/test_animation-starttime.html |  - : Elided 3 passes or known failures.
[task 2018-01-11T12:02:41.230Z] 12:02:41     INFO -  Buffered messages finished
[task 2018-01-11T12:02:41.231Z] 12:02:41     INFO -  179 INFO TEST-UNEXPECTED-FAIL | dom/animation/test/css-animations/test_animation-starttime.html | Sanity test to check round-tripping assigning to a new animation's startTime - Sanity test to check round-tripping assigning to a new animation's startTime: assert_approx_equals: Check setting of startTime actually works expected 2463.16 +/- 0.0005 but got 2463.14
[task 2018-01-11T12:02:41.235Z] 12:02:41     INFO -  180 INFO TEST-PASS | dom/animation/test/css-animations/test_animation-starttime.html |  - : Elided 17 passes or known failures.
[task 2018-01-11T12:02:41.237Z] 12:02:41     INFO -  181 INFO TEST-OK | dom/animation/test/css-animations/test_animation-starttime.html | took 11423ms
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
That push/failure didn't have this patch in it :)
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Whiteboard: [stockwell fixed:timing]
Without timer clamping enabled, this seems like a new failure.
Assignee: tom → nobody
Assignee: nobody → tom
You need to log in before you can comment on or make changes to this bug.