Closed Bug 503681 Opened 11 years ago Closed 11 years ago

browser_bug134911.js times out intermittently

Categories

(Core :: Layout: Form Controls, defect)

x86
Linux
defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla1.9.2a1

People

(Reporter: ehsan, Assigned: smontagu)

References

()

Details

(Keywords: intermittent-failure, verified1.9.0.14, verified1.9.1.1)

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1247329235.1247335617.25038.gz
Linux mozilla-central unit test on 2009/07/11 09:20:35

The previous run was from a different changeset <http://hg.mozilla.org/mozilla-central/rev/20be4e9999cf> but the only changesets in the range is the backout of  a font change, which can't be relevant here; so this should be intermittent.
Whiteboard: [orange]
The test uses a setTimeout (twice, acually).  Why?  It looks to me like if either timeout is too short (which it could easily be on a loaded machine) the test will throw and never reach finish()...  Why are we not observing events instead of relying on timeouts here?
Blocks: 134911
(In reply to comment #1)
> The test uses a setTimeout (twice, acually).  Why?

Because I knew no better at the time?

Fixed to use event listeners both in browser_bug134911.js and the very similar browser_bug92473.js, pushed in http://hg.mozilla.org/mozilla-central/rev/b9012a18d2c6

A possible question mark is that I use the load event both for the original page load and for changing charset. I'm not sure if there is any other event that would work instead, and I'm not sure if there is a guarantee that changing charset will fire a load event, e.g. if we fix bug 61363.
Assignee: nobody → smontagu
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/d08559185681
Status: NEW → RESOLVED
Closed: 11 years ago
Keywords: fixed1.9.1.1
Resolution: --- → FIXED
Checking in docshell/test/browser/browser_bug134911.js;
/cvsroot/mozilla/docshell/test/browser/browser_bug134911.js,v  <--  browser_bug134911.js
new revision: 1.2; previous revision: 1.1
done
Checking in docshell/test/browser/browser_bug92473.js;
/cvsroot/mozilla/docshell/test/browser/browser_bug92473.js,v  <--  browser_bug92473.js
new revision: 1.2; previous revision: 1.1
Keywords: fixed1.9.0.13
Flags: in-testsuite+
Target Milestone: --- → mozilla1.9.2a1
> and I'm not sure if there is a guarantee that changing charset will fire a load
> event, e.g. if we fix bug 61363.

If we change that, this test will just fail reliably, right?  We can worry about it then.
Simon: can you verify that this is resolved for mozilla-1.9.1?
I verify that there are no SetTimeouts in http://mxr.mozilla.org/mozilla1.9.1/source/docshell/test/browser/browser_bug134911.js. Not sure what other verification is possible in this case.
Status: RESOLVED → VERIFIED
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.