Closed
Bug 1311588
Opened 8 years ago
Closed 8 years ago
Intermittent http://10.0.2.2:8854/tests/layout/reftests/web-animations/1267937-1.html == http://10.0.2.2:8854/tests/layout/reftests/web-animations/1267937-ref.html | (LOAD ONLY), max difference: 255, number of differing pixels: 20000
Categories
(Core :: DOM: Animation, defect, P3)
Core
DOM: Animation
Tracking
()
RESOLVED
FIXED
mozilla52
People
(Reporter: intermittent-bug-filer, Assigned: hiro)
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
Filed by: philringnalda [at] gmail.com https://treeherder.mozilla.org/logviewer.html#?job_id=1824370&repo=mozilla-beta https://archive.mozilla.org/pub/mobile/tinderbox-builds/mozilla-beta-android-api-15-debug/1476903622/mozilla-beta_ubuntu64_vm_armv7_large-debug_test-plain-reftest-46-bm121-tests1-linux64-build3.txt.gz https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://archive.mozilla.org/pub/mobile/tinderbox-builds/mozilla-beta-android-api-15-debug/1476903622/mozilla-beta_ubuntu64_vm_armv7_large-debug_test-plain-reftest-46-bm121-tests1-linux64-build3.txt.gz&only_show_unexpected=1
Assignee | ||
Comment 1•8 years ago
|
||
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.
Assignee | ||
Comment 2•8 years ago
|
||
I don't remember why I did not use steps(1, start)...
Assignee | ||
Updated•8 years ago
|
Component: Layout → DOM: Animation
Priority: -- → P3
Assignee | ||
Comment 3•8 years ago
|
||
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 hidden (mozreview-request) |
Comment 5•8 years ago
|
||
mozreview-review |
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+
Pushed by hiikezoe@mozilla-japan.org: https://hg.mozilla.org/integration/autoland/rev/7eb011a5b421 Use longer duration to avoid failure on Android. r=boris
Comment 8•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7eb011a5b421
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Comment 9•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/39299d2a7f1a
status-firefox51:
--- → fixed
Comment 10•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/1ad4ab26875c
status-firefox50:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•