596237 - [SeaMonkey 2.1] TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/content/html/document/test/browser_bug592641.js

Status: VERIFIED FIXED

(SeaMonkey :: Testing Infrastructure, defect)

Description

[Bruno 'Aqualon' Escherl]

http://tinderbox.mozilla.org/showlog.cgi?tree=SeaMonkey&errorparser=unittest&logfile=1284472891.1284476159.31409.gz&buildtime=1284472891&buildname=Linux comm-central-trunk debug test mochitest-other&fulltext=1#err1

This fails after the test should be finished already. Perhaps similar issue to bug 595188?

Comment 1

[[e-mail]]

Exactly the same problem... I hadn't noticed because I ran the tests locally on a debug build and the debug spew hid the unexpected fails, and because they happened after the test shutdown the harness claimed that the test passed :-(

Comment 2

[[e-mail]]

Attachment: Remove event listeners

Comment 3

[Bobby Holley (:bholley)]

Comment on attachment 475115 [details] [diff] [review]
Remove event listeners

Looks fine - r=bholley.

I'm curious why this is necessary though - those tabs are removed, so their listeners should go away, no?

Comment 4

[[e-mail]]

(In reply to comment #3)
> I'm curious why this is necessary though - those tabs are removed, so their
> listeners should go away, no?
Unfortunately SeaMonkey's undo close tab keeps the browser around so while you can undo an accidental close tab very quickly a quirk is that it loads about:blank in the browser too, thus causing unexpected extra load events.

Comment 5

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/c36b84b97484

Comment 6

[Bruno 'Aqualon' Escherl]

Don't see this failure anymore on our tinderboxes.

-> VERIFIED