494113 - crashtest: Intermittent "441785-1.xul | timed out waiting for reftest-wait to be removed (after onload fired)"

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Jim Mathies [:jimm]]

REFTEST TEST-UNEXPECTED-FAIL | file:///e:/builds/moz2_slave/mozilla-central-win32-unittest/build/content/xul/templates/src/crashtests/441785-1.xul | timed out waiting for reftest-wait to be removed (after onload fired)
REFTEST TEST-UNEXPECTED-FAIL | file:///e:/builds/moz2_slave/mozilla-central-win32-unittest/build/content/xul/templates/src/crashtests/441785-1.xul | timed out waiting for reftest-wait to be removed (after onload fired)

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1242867229.1242873251.1579.gz

Disable on 1.9.1, but still showing up on trunk. Not sure if this is the right component.

Comment 1

[Mike Beltzner [:beltzner, not reading bugmail]]

Again on Linux mozilla-central unit test on 2009/05/20 18:53:28

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1242870808.1242877506.8553.gz#err0

Comment 2

[Robert Strong (they/them - no direct email)]

Linux mozilla-central unit test on 2009/05/22 12:01:46
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1243018906.1243025593.32268.gz&fulltext=1

Comment 4

[Joe Drew (not getting mail)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1243557923.1243564289.10029.gz

Comment 5

[Neil Deakin]

The test is failing because it isn't waiting for the datasource to load before checking the results.

Comment 6

[Jesse Ruderman]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1243730397.1243738414.27738.gz

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1243706680.1243711882.14689.gz

Comment 7

[Serge Gautherie (:sgautherie)]

(In reply to comment #3)
> *** Bug 495077 has been marked as a duplicate of this bug. ***

Note that bug has a (security) dependency I can't move here...

Comment 8

[Boris Zbarsky [:bzbarsky]]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1243882732.1243887630.18906.gz

Comment 9

[Olli Pettay [:smaug][[e-mail]]]

(In reply to comment #5)
> The test is failing because it isn't waiting for the datasource to load before
> checking the results.
How do I make it wait until datasource is loaded?
I tried to do that with my changes to the testcase in bug 492037.
The original patch failed even more often.

Comment 10

[Neil Deakin]

(In reply to comment #9)
> (In reply to comment #5)
> > The test is failing because it isn't waiting for the datasource to load before
> > checking the results.
> How do I make it wait until datasource is loaded?
> I tried to do that with my changes to the testcase in bug 492037.
> The original patch failed even more often.

Call GetDataSourceBlocking on the datasource uri beforehand as in the test from bug 495648. I tried to do this but it seems crashtests don't have enough privileges.

Comment 11

[David Bolter [:davidb] (NeedInfo me for attention)]

http://tinderbox.mozilla.org/showlog.cgi?log=MozillaTry/1244468190.1244474868.15487.gz

Comment 12

[Daniel Holbert [:dholbert]]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1244473503.1244479386.22946.gz
Linux mozilla-central unit test on 2009/06/08 08:05:03
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1244480703.1244487343.4767.gz
Linux mozilla-central unit test on 2009/06/08 10:05:03

Comment 13

[Olli Pettay [:smaug][[e-mail]]]

(In reply to comment #10) 
>I tried to do this but it seems crashtests don't have enough
> privileges.
So the test shouldn't be a crashtest but mochitest.

Comment 14

[Neil Deakin]

Attachment: move test and load datasource before test runs

Comment 15

[Ben Newman (:bnewman) (:benjamn)]

http://tinderbox.mozilla.org/showlog.cgi?log=MozillaTry/1244583178.1244589680.15444.gz
Linux try hg unit test on 2009/06/09 14:32:58

Comment 16

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 382312 [details] [diff] [review]
move test and load datasource before test runs

Perhaps you could test this on tryserver first. Then land to trunk.

Comment 17

[Neil Deakin]

(In reply to comment #16)
> (From update of attachment 382312 [details] [diff] [review])
> Perhaps you could test this on tryserver first. Then land to trunk.

Unfortunately the tryserver doesn't handle patches that include file moves. I suppose I could make a special file duplicating patch, but since the test only fails occasionally it seems like it might not be worth it.

Comment 18

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1244734033.1244740016.6753.gz
Linux mozilla-central unit test on 2009/06/11 08:27:13

Comment 19

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1244747510.1244754764.5940.gz
Linux mozilla-central unit test on 2009/06/11 12:11:50

Comment 20

[Neil Deakin]

Checked this in. Reopen if this occurs again.