Closed Bug 1476950 Opened 6 years ago Closed 5 years ago

Intermittent dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed - got 1, expected +0

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox-esr60 --- unaffected
firefox-esr68 --- unaffected
firefox68 --- unaffected
firefox69 --- unaffected
firefox70 --- fixed

People

(Reporter: RaulG, Assigned: tjr)

References

(Regression)

Details

(Keywords: intermittent-failure, regression, Whiteboard: [retriggered])

[Tracking Requested - why for this release]:

Central as beta simulation: https://treeherder.mozilla.org/#/jobs?repo=try&revision=20a04997cec50341d5f8fc16b9d1a9e981c0c94f

Log link: https://treeherder.mozilla.org/logviewer.html#?job_id=188924461&repo=try&lineNumber=3095

Log snippet: 

04:32:34     INFO - TEST-START | dom/animation/test/mozilla/test_restyles.html
04:32:36     INFO - TEST-INFO | started process screencapture
04:32:36     INFO - TEST-INFO | screencapture: exit 0
04:32:36     INFO - Buffered messages logged at 04:32:34
04:32:36     INFO - AddTask.js | Entering test restyling_for_main_thread_animations
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | undefined assertion name 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | CSS animations running on the main-thread should update style on the main thread 
04:32:36     INFO - AddTask.js | Leaving test restyling_for_main_thread_animations
04:32:36     INFO - AddTask.js | Entering test no_restyling_for_compositor_animations
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | undefined assertion name 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | CSS animations running on the compositor should not update style on the main thread 
04:32:36     INFO - AddTask.js | Leaving test no_restyling_for_compositor_animations
04:32:36     INFO - AddTask.js | Entering test no_restyling_for_compositor_transitions
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | undefined assertion name 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | CSS transitions running on the compositor should not update style on the main thread 
04:32:36     INFO - Buffered messages logged at 04:32:35
04:32:36     INFO - AddTask.js | Leaving test no_restyling_for_compositor_transitions
04:32:36     INFO - AddTask.js | Entering test no_restyling_when_animation_duration_is_changed
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | undefined assertion name 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Animations running on the compositor should not update style on the main thread 
04:32:36     INFO - AddTask.js | Leaving test no_restyling_when_animation_duration_is_changed
04:32:36     INFO - AddTask.js | Entering test only_one_restyling_after_finish_is_called
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | undefined assertion name 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Animations running on the compositor should only update style once after finish() is called 
04:32:36     INFO - TEST-FAIL | dom/animation/test/mozilla/test_restyles.html | Bug 1415457: Animations running on the compositor should only update style once after finish() is called - got 1, expected +0
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Finished animations should never update style after one restyle happened for finish() 
04:32:36     INFO - AddTask.js | Leaving test only_one_restyling_after_finish_is_called
04:32:36     INFO - AddTask.js | Entering test no_restyling_mouse_movement_on_finished_transition
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Finished transitions should restyle once after Animation.finished was fulfilled 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Finished transitions should never cause restyles when mouse is moved on the transitions 
04:32:36     INFO - AddTask.js | Leaving test no_restyling_mouse_movement_on_finished_transition
04:32:36     INFO - AddTask.js | Entering test no_restyling_mouse_movement_on_finished_animation
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Finished animations should restyle once after Animation.finished was fulfilled 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Finished animations should never cause restyles when mouse is moved on the animations 
04:32:36     INFO - AddTask.js | Leaving test no_restyling_mouse_movement_on_finished_animation
04:32:36     INFO - AddTask.js | Entering test no_restyling_compositor_animations_out_of_view_element
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | undefined assertion name 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Animations running on the compositor in an out-of-view element should never cause restyles 
04:32:36     INFO - AddTask.js | Leaving test no_restyling_compositor_animations_out_of_view_element
04:32:36     INFO - AddTask.js | Entering test no_restyling_main_thread_animations_out_of_view_element
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Animations running on the main-thread in an out-of-view element should never cause restyles 
04:32:36     INFO - AddTask.js | Leaving test no_restyling_main_thread_animations_out_of_view_element
04:32:36     INFO - AddTask.js | Entering test no_restyling_compositor_animations_in_scrolled_out_element
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Animations running on the compositor for elements which are scrolled out should never cause restyles 
04:32:36     INFO - Buffered messages logged at 04:32:36
04:32:36     INFO - AddTask.js | Leaving test no_restyling_compositor_animations_in_scrolled_out_element
04:32:36     INFO - AddTask.js | Entering test no_restyling_missing_keyframe_opacity_animations_on_scrolled_out_element
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Opacity animations on scrolled out elements should never cause restyles even if the animation has missing keyframes 
04:32:36     INFO - AddTask.js | Leaving test no_restyling_missing_keyframe_opacity_animations_on_scrolled_out_element
04:32:36     INFO - AddTask.js | Entering test restyling_transform_animations_in_scrolled_out_element
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | The transform animation is not running on the compositor 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - TEST-PASS | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed 
04:32:36     INFO - Buffered messages finished
04:32:36     INFO - TEST-UNEXPECTED-FAIL | dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed - got 1, expected +0
04:32:36     INFO -     SimpleTest.is@SimpleTest/SimpleTest.js:312:5
04:32:36     INFO -     restyling_transform_animations_in_scrolled_out_element@dom/animation/test/mozilla/file_restyles.html:402:9
04:32:36     INFO - Not taking screenshot here: see the one that was previously logged
Summary: Intermittent dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed - got 1, expected +0 when Gecko 63 merges to Beta on 2018-08-27 → Intermittent dom/animation/test/mozilla/test_restyles.html | Transform animation running on the element which is scrolled out should be throttled until 200ms is elapsed - got 1, expected +0
See Also: → 1387894
Assignee: nobody → tom

I'm hopefully going to fix this in Bug 1387894

Hi Tom.
I filed today bug 1571226 and bug 1571227 and I believe they are regressions from here https://hg.mozilla.org/integration/autoland/rev/e73d900b1dcf01da7c7023d7d6ee70043be5d9cf can you please take a look?

Flags: needinfo?(tom)
Regressions: 1571226
Regressions: 1571227
Regressions: 1571038

Joel do we backout?

Flags: needinfo?(jmaher)

I'll work on the intermittents Monday. I'd prefer not to back out - this patch is likely to produce intermittents which I'll work on. If we back it out, I'll fix these three, we'll reland, and it might cause more, and then we'll be in this cycle of backing out/relanding.

Flags: needinfo?(tom)
Regressed by: 1387894

This particular intermittent is fixed (AFAICT). The failures logged in Comment #30 are all either Bug 1571227 or Bug 1571038

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.