The default bug view has changed. See this FAQ.

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

VERIFIED FIXED in seamonkey2.1b1

Status

SeaMonkey
Testing Infrastructure
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: Bruno 'Aqualon' Escherl, Assigned: neil@parkwaycc.co.uk)

Tracking

(Blocks: 1 bug)

Trunk
seamonkey2.1b1
x86_64
Linux
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
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?
(Assignee)

Comment 1

7 years ago
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 :-(
(Assignee)

Comment 2

7 years ago
Created attachment 475115 [details] [diff] [review]
Remove event listeners
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #475115 - Flags: review?(bobbyholley+bmo)
(Assignee)

Updated

7 years ago
Depends on: 592641
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?
Attachment #475115 - Flags: review?(bobbyholley+bmo) → review+
(Assignee)

Comment 4

7 years ago
(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.
http://hg.mozilla.org/mozilla-central/rev/c36b84b97484
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b1
(Reporter)

Comment 6

7 years ago
Don't see this failure anymore on our tinderboxes.

-> VERIFIED
Status: RESOLVED → VERIFIED
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.