Intermittent image/test/mochitest/test_discardAnimatedImage.html | called finish() multiple times

RESOLVED FIXED in Firefox 55

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: intermittent-bug-filer, Unassigned)

Tracking

({intermittent-failure})

unspecified
mozilla55
intermittent-failure
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

The test needs a simple bool check to avoid calling finish more than once. I'll push a patch in the morning. If it becomes too frequent you can temporarily disable the test until then.

Comment 2

2 years ago
Pushed by tnikkel@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/17d681fd2b32
Fix test_discardAnimatedImage.html to not call finish multiple times.

Comment 3

2 years ago
Pushed by tnikkel@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/415cd9f87458
Don't record further successful results after finishing in test_discardAnimatedImage.html.
Backed out both in https://hg.mozilla.org/integration/mozilla-inbound/rev/21062cd63dd7 since both the first alone and the first plus the second resulted in a 50% chance of one or more "got frame update - Result logged after SimpleTest.finish()".
Okay, I'll stop wasting the sheriffs time. try server of what should be a proper fix

https://treeherder.mozilla.org/#/jobs?repo=try&revision=1ddd796e90821490b6fbf93866534d210fd5e34c

Until I get the results of that I'm going to disable this test and repush bug 686905. Since the test is passing its just a bookkeeping issue of the test calling ok(true,...) after the test is over.
Keywords: leave-open

Comment 6

2 years ago
Pushed by tnikkel@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e464cda66cf6
Fix test_discardAnimatedImage.html to not call finish multiple times.
https://hg.mozilla.org/integration/mozilla-inbound/rev/fc67e6aa71ea
Don't record further successful results after finishing in test_discardAnimatedImage.html.
https://hg.mozilla.org/integration/mozilla-inbound/rev/8d01060abf16
Do a better job of not recording successful results after test finish in test_discardAnimatedImage.html.
https://hg.mozilla.org/integration/mozilla-inbound/rev/008043649fc8
Disable test_discardAnimatedImage.html temporarily.

Comment 8

2 years ago
Pushed by tnikkel@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e1e3401303c8
Enable test_discardAnimatedImage.html.
Keywords: leave-open
You need to log in before you can comment on or make changes to this bug.