Serge Gautherie (:sgautherie) Reporter
	Description • 15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1251632618.1251638510.5559.gz
Linux mozilla-central unit test on 2009/08/30 04:43:38
{
Running chrome://mochikit/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgrdlg.js...
TEST-UNEXPECTED-FAIL | ... | Timed out
}

	Paul O'Shannessy [:zpao] (not bugmail, email directly)
	Comment 1 • 15 years ago

I don't think this is a dupe of bug 513560 but browser_passwordmgrdlg.js was changed in that bug, so worth a mention.

	(no longer active) Assignee
	Comment 2 • 15 years ago

(In reply to comment #1)
> I don't think this is a dupe of bug 513560 but browser_passwordmgrdlg.js was
> changed in that bug, so worth a mention.

Bug 513560 is this bug, right?

	(no longer active) Assignee
	Comment 3 • 15 years ago

This could be caused by browser_passwordmgr_sort.js finishing before the passwordmgr dialog has actually closed, and browser_passwordmgr.js timing out looking for a domwindowopened.

In this fix, I also ported the focus handling rewrite changes to browser_passwordmgr_sort.js as well.

	Justin Dolske [:Dolske]
	Comment 4 • 15 years ago

Comment on attachment 398877 [details] [diff] [review]
Patch (v1)

Unlucky you, Enn just blogged about what should be a better way to fix this:

http://enndeakin.wordpress.com/2009/09/08/how-to-fix-focus-related-problems-in-tests/

	Marco Bonardo [:mak]
	Comment 5 • 15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1252904272.1252907789.17847.gz
Linux mozilla-central test everythingelse on 2009/09/13 21:57:52

	(no longer active) Assignee
	Comment 6 • 15 years ago

(In reply to comment #4)
> (From update of attachment 398877 [details] [diff] [review])
> Unlucky you, Enn just blogged about what should be a better way to fix this:
> 
> http://enndeakin.wordpress.com/2009/09/08/how-to-fix-focus-related-problems-in-tests/

I filed bug 516440 to port this to the browser chrome tests framework, so that I can use it here.

	Paul O'Shannessy [:zpao] (not bugmail, email directly)
	Comment 7 • 15 years ago

(In reply to comment #2)
Oh yea, my mistake. I must have copied the wrong bug number... Maybe the one that you fixed before?

	David Bolter [:davidb] (NeedInfo me for attention)
	Comment 8 • 15 years ago

(happend on branch)
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.6/1253719685.1253728275.32169.gz
Linux mozilla-1.9.2 unit test on 2009/09/23 08:28:05

	David Baron :dbaron:
	Comment 9 • 15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1255645695.1255649616.21267.gz
Linux mozilla-central test everythingelse on 2009/10/15 15:28:15  
TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgrdlg.js | Timed out

and then a possibly-related error on shutdown:

buildbot.slave.commands.TimeoutError: command timed out: 1200 seconds without output, killing pid 4625

	(not currently active) Ted Mielczarek
	Comment 10 • 15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1257932979.1257938217.8708.gz
Linux mozilla-central debug test everythingelse on 2009/11/11 01:49:39  
s: moz2-linux-slave27

	(no longer active) Assignee
	Comment 11 • 15 years ago

Switch to waitForFocus as well as the previous fix for waiting until the window is closed.

	Paul O'Shannessy [:zpao] (not bugmail, email directly)
	Comment 12 • 15 years ago

FYI, waitForFocus can be a bit finicky in browser-chrome tests. There's a fix checked in, but it doesn't always work. See bug 521233.

	(no longer active) Assignee
	Comment 13 • 15 years ago

Thanks for mentioning this, I added that as a dependency on this bug, though actually the other part of the patch is what really solves this orange.

	(no longer active) Assignee
	Comment 14 • 15 years ago

http://hg.mozilla.org/mozilla-central/rev/5b8c9b073078

	(no longer active) Assignee
	Comment 15 • 15 years ago

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/beac3ae9eef0