1310096 - Intermittent http://10.0.2.2:8854/tests/layout/reftests/css-animations/no-stacking-context-transform-removing-animation-in-delay.html | load failed: timed out waiting for reftest-wait to be removed

Status: RESOLVED FIXED

(Core :: DOM: Animation, defect)

Description

[Treeherder Bug Filer]

Filed by: philringnalda [at] gmail.com

https://treeherder.mozilla.org/logviewer.html#?job_id=37576992&repo=mozilla-inbound

https://queue.taskcluster.net/v1/task/cPOqUzEbQOCzqPTmt7nEZQ/runs/0/artifacts/public%2Flogs%2Flive_backing.log

https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://queue.taskcluster.net/v1/task/cPOqUzEbQOCzqPTmt7nEZQ/runs/0/artifacts/public%2Flogs%2Flive_backing.log&only_show_unexpected=1

Comment 1

[Intermittent Failures Robot]

6 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 4
* autoland: 2

Platform breakdown:
* android-4-3-armv7-api15: 6

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1310096&startday=2016-10-10&endday=2016-10-16&tree=all

Comment 2

[Hiroyuki Ikezoe (:hiro)]

I could reproduce this failure locally on Android emulator.  It seems that on Android the second MozAfterPaint was not received (while GC was processed?).
 
Replacing the second wait with requestAnimationFrame seems to work.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=be246bd9c4b8&exclusion_profile=false

Comment 3

[Hiroyuki Ikezoe (:hiro)]

Try results for desktop seem to good.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ea73f95da76b&exclusion_profile=false

Comment 4

[Hiroyuki Ikezoe (:hiro)]

Attachment: Bug 1310096 - Replace the wait for the second MozAfterPaint with rAF callback to avoid intermittent failure on Android.

Review commit: https://reviewboard.mozilla.org/r/86802/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/86802/

Comment 5

[Hiroyuki Ikezoe (:hiro)]

Boris, could you please review this since Brian is away.  Thanks for reviewing in these days!

Comment 6

[Boris Chiou [:boris]]

Comment on attachment 8802405 [details]
Bug 1310096 - Replace the wait for the second MozAfterPaint with rAF callback to avoid intermittent failure on Android.

https://reviewboard.mozilla.org/r/86802/#review85820

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/266e70286889
Replace the wait for the second MozAfterPaint with rAF callback to avoid intermittent failure on Android. r=boris

Comment 8

[Hiroyuki Ikezoe (:hiro)]

Thanks!

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/266e70286889

Comment 10

[Intermittent Failures Robot]

10 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 5
* autoland: 3
* try: 1
* fx-team: 1

Platform breakdown:
* android-4-3-armv7-api15: 10

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1310096&startday=2016-10-17&endday=2016-10-23&tree=all

Comment 11

[Hiroyuki Ikezoe (:hiro)]

Closing because there is no new orange after landing.  There is a failure <https://treeherder.mozilla.org/logviewer.html#?repo=try&job_id=29455918#L2752> on try after landing, but the changeset does not include this fix.