Closed Bug 439164 Opened 16 years ago Closed 15 years ago

[FIXr]intermittent mochitest failure in test_bug413310


(Core :: DOM: Navigation, defect)

Not set





(Reporter: joduinn, Assigned: bzbarsky)



(Keywords: fixed1.9.0.2, intermittent-failure)


(1 file)

We're investigating intermittent test failures, and find that this test
sometimes passes, sometimes fails, even when there are no code changes. Its
happened with a few builds this week. (It was happening before, as well as
after, the recent NetApp woes). 

Can you review how this test was written and see if the test can be made more

suite: mochitest
stdio line#13065
error msg: *** 13678 ERROR FAIL Error thrown during test: doNextStep is not defined /tests/docshell/test/test_bug413310.html	

Note: This test does not *break* builds, they only fail out as orange.
Hmm.  Yeah, I think I see how this could happen.  Diff that should help coming up.
Attached patch Like sSplinter Review
The comments more or less explain what the change is here.
Attachment #325131 - Flags: review?(joduinn)
John - is the patch good to go?
Boris: this should probably be reviewed by Waldo (who's afk), sayrer or myself. R+ from me though you should add a space to the comment line (NOTget):

+// modified accordingly!  It assumes that subframes do NOTget bfcached.

I'm not sure this will fix the problem as we're still using explicit timeout values here, but the order might improve reliability.

Also, as a final note, it's well-past May now. Might want to change your "very little time" message. Helpful tip provided at no extra charge!
Attachment #325131 - Flags: review?(joduinn) → review+
Minor nit: can you fix typo in comment "do NOTget " should be "do NOT get "! 
> I'm not sure this will fix the problem as we're still using explicit timeout
> values here

That's OK.  All that's needed there is that the code run async.  When is not important.  I almost changed the timeout value to 0 to make that clearer.

And fixed the name.  If forgot to update it when late May didn't free up...

I won't be able to check this in for probably a week or so.  If someone can get to it before then, great.
Assignee: nobody → bzbarsky
Keywords: checkin-needed
Summary: intermittent mochitest failure in test_bug413310 → [FIXr]intermittent mochitest failure in test_bug413310
Version: unspecified → Trunk
Gavin or reed can you land this?
Landed on mozilla-central:
and CVS trunk:
mozilla/docshell/test/test_bug413310.html 	1.2 
Keywords: checkin-needed
Should be fixed.
Closed: 15 years ago
Resolution: --- → FIXED
Whiteboard: [orange]
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.