Closed Bug 1226193 Opened 6 years ago Closed 6 years ago

test_animation_observers.html times out on try with C++APZ enabled

Categories

(Core :: DOM: Animation, defect)

Unspecified
Android
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed

People

(Reporter: rbarker, Assigned: rbarker)

References

Details

Attachments

(1 file)

The test_animation_observers.html is consistently timing out on try when C++APZ in enabled in Fennec. Test passes when run locally.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=4895d3e1522e
Assignee: nobody → rbarker
Blocks: 1207748
Attachment #8689531 - Flags: review?(bugs)
Attachment #8689531 - Flags: review?(bugs) → review?(cam)
Comment on attachment 8689531 [details] [diff] [review]
0001-Bug-1226193-test_animation_observers.html-times-out-on-try-with-C-APZ-enabled-15111909-59d3e2d.patch

Review of attachment 8689531 [details] [diff] [review]:
-----------------------------------------------------------------

I'd be surprised if the test needs just a bit more time, rather than there being a bug in the test that causes it to wait indefinitely, but let's try.
Attachment #8689531 - Flags: review?(cam) → review+
We did do additional try pushes with this patch and it seemed to resolve the failures we were seeing before. Thanks for the review!
https://hg.mozilla.org/mozilla-central/rev/b140ad795489
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in before you can comment on or make changes to this bug.