Last Comment Bug 721589 - Intermittent M4 failure: /tests/image/test/mochitest/test_error_events.html | Got onload for error-late
: Intermittent M4 failure: /tests/image/test/mochitest/test_error_events.html |...
Status: RESOLVED FIXED
: intermittent-failure
Product: Core
Classification: Components
Component: ImageLib (show other bugs)
: unspecified
: x86 Mac OS X
: -- normal (vote)
: mozilla12
Assigned To: Justin Lebar (not reading bugmail)
:
:
Mentors:
Depends on:
Blocks: 438871 715308
  Show dependency tree / graph
 
Reported: 2012-01-26 16:55 PST by Justin Lebar (not reading bugmail)
Modified: 2012-11-25 19:31 PST (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Attachments
Patch v1 (remove large image from test) (68.15 KB, patch)
2012-01-27 09:23 PST, Justin Lebar (not reading bugmail) 		joe: review+
Details | Diff | Splinter Review
View All Add an attachment (proposed patch, testcase, etc.)

Description Justin Lebar (not reading bugmail) 2012-01-26 16:55:22 PST 
There must be a race condition here which causes us to fire either an onerror or an onload for this erroneous image depending on how much stuff we decode.

https://tbpl.mozilla.org/php/getParsedLog.php?id=8863420&tree=Mozilla-Inbound
Comment 1 Treeherder Robot 2012-01-26 16:58:32 PST 
mbrubeck
https://tbpl.mozilla.org/php/getParsedLog.php?id=8863420&tree=Mozilla-Inbound
Rev3 WINNT 5.1 mozilla-inbound opt test mochitests-4/5 on 2012-01-26 15:58:08

235 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_error_events.html | Got onload for error-late
Comment 2 Treeherder Robot 2012-01-27 07:26:03 PST 
philor
https://tbpl.mozilla.org/php/getParsedLog.php?id=8879394&tree=Mozilla-Inbound
Rev3 WINNT 5.1 mozilla-inbound pgo test mochitests-4/5 on 2012-01-27 06:34:34

234 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_error_events.html | Got onload for error-late
Comment 3 Justin Lebar (not reading bugmail) 2012-01-27 09:23:28 PST 
Created attachment 592160 [details] [diff] [review]
Patch v1 (remove large image from test)

Review: JOEDREW
There's clearly a race here with respect to how much of this image we decode in one timeslice.
Comment 4 Joe Drew (not getting mail) 2012-01-27 11:00:26 PST 
Comment on attachment 592160 [details] [diff] [review]
Patch v1 (remove large image from test)

:/
Comment 5 Treeherder Robot 2012-01-28 11:08:46 PST 
philor
https://tbpl.mozilla.org/php/getParsedLog.php?id=8909119&tree=Fx-Team
Rev3 WINNT 5.1 fx-team pgo test mochitests-4/5 on 2012-01-28 09:11:03

235 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_error_events.html | Got onload for error-late
Comment 6 Treeherder Robot 2012-01-28 17:12:05 PST 
zackw%panix.com
https://tbpl.mozilla.org/php/getParsedLog.php?id=8913697&tree=Firefox
Rev3 WINNT 5.1 mozilla-central opt test mochitests-4/5 on 2012-01-28 16:58:54

235 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_error_events.html | Got onload for error-late
Comment 7 Joe Drew (not getting mail) 2012-01-28 19:07:13 PST 
https://hg.mozilla.org/mozilla-central/rev/9a36661ced48
Comment 8 Treeherder Robot 2012-01-29 09:33:20 PST 
philor
https://tbpl.mozilla.org/php/getParsedLog.php?id=8923002&tree=Fx-Team
Rev3 WINNT 5.1 fx-team pgo test mochitests-4/5 on 2012-01-29 09:20:59

234 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_error_events.html | Got onload for error-late
Note You need to log in before you can comment on or make changes to this bug.