Closed Bug 1311588 Opened 8 years ago Closed 8 years ago

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox50 --- fixed
firefox51 --- fixed
firefox52 --- fixed

People

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

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

The animations was rendered at 400px position, that means the current time of animation was greater than 50s (The animation's timing function is "steps(2, start)".). 

13:44:32     INFO -  REFTEST TEST-LOAD | http://10.0.2.2:8854/tests/layout/reftests/web-animations/1267937-1.html | 43 / 376 (11%)
13:45:35     INFO -  REFTEST INFO | RESTORE PREFERENCE pref(dom.animations-api.core.enabled,false)

The test actually took 63s, we need a longer duration for this animation.
I don't remember why I did not use steps(1, start)...
Component: Layout → DOM: Animation
Priority: -- → P3
I've decided to make the duration longer because using steps(1, start) needs to modify the reference too.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=065a7650396c&exclusion_profile=false&selectedJob=29525957
Comment on attachment 8802808 [details]
Bug 1311588 - Use longer duration to avoid failure on Android.

https://reviewboard.mozilla.org/r/87108/#review86126

Longer duration makes sense, according to your investigation. Thanks.
Attachment #8802808 - Flags: review?(boris.chiou) → review+
Thanks!
Assignee: nobody → hiikezoe
Status: NEW → ASSIGNED
Pushed by hiikezoe@mozilla-japan.org:
https://hg.mozilla.org/integration/autoland/rev/7eb011a5b421
Use longer duration to avoid failure on Android. r=boris
https://hg.mozilla.org/mozilla-central/rev/7eb011a5b421
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: