516372 - random failure in test_bug322588.html | unexpected events - got " unload", expected " unload load"

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

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

18931 INFO Running /tests/content/events/test/test_bug322588.html...
18932 ERROR TEST-UNEXPECTED-FAIL | /tests/content/events/test/test_bug322588.html | unexpected events - got " unload", expected " unload load"

Comment 1

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.6-Unittest/1253580730.1253583409.2229.gz
Linux mozilla-1.9.2 test mochitests on 2009/09/21 17:52:10

Comment 2

[Shawn Wilsher :sdwilsh]

OS X 10.5.2 mozilla-central test debug mochitests-1/5 on 2009/10/22 14:58:06  
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256248686.1256250386.15572.gz#err15

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256317429.1256319629.15833.gz
WINNT 5.2 mozilla-central test debug mochitests-1/5 on 2009/10/23 10:03:49  
29713 ERROR TEST-UNEXPECTED-FAIL | /tests/content/events/test/test_bug322588.html | unexpected events - got "", expected " unload load"

Comment 4

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256466706.1256468162.9264.gz
Linux mozilla-central test debug mochitests-1/5 on 2009/10/25 03:31:46  
29673 ERROR TEST-UNEXPECTED-FAIL | /tests/content/events/test/test_bug322588.html | unexpected events - got "", expected " unload load"

Comment 5

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256493791.1256494770.5411.gz
WINNT 5.2 mozilla-central test debug mochitests-1/5 on 2009/10/25 11:03:11

Comment 6

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256494921.1256496253.22175.gz
WINNT 5.2 mozilla-central test debug mochitests-1/5 on 2009/10/25 11:22:01

Comment 7

[Makoto Kato [:m_kato]]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256551331.1256553469.26515.gz

Comment 8

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256542852.1256544583.23084.gz
Linux mozilla-central test debug mochitests-1/5 on 2009/10/26 00:40:52

Comment 9

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256582901.1256584926.10835.gz
WINNT 5.2 mozilla-central test debug mochitests-1/5 on 2009/10/26 11:48:21

Comment 10

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256589850.1256591457.18741.gz
Linux mozilla-central test debug mochitests-1/5 on 2009/10/26 13:44:10

Comment 11

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256595430.1256596919.15558.gz
Linux mozilla-central test debug mochitests-1/5 on 2009/10/26 15:17:10

Comment 12

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256611843.1256613361.969.gz
Linux mozilla-central test debug mochitests-1/5 on 2009/10/26 19:50:43

Comment 13

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256640773.1256642067.26269.gz
WINNT 5.2 mozilla-central test debug mochitests-1/5 on 2009/10/27 03:52:53

Comment 14

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256663596.1256665095.10374.gz
Linux mozilla-central test debug mochitests-1/5 on 2009/10/27 10:13:16

Comment 15

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256665204.1256666283.23623.gz
WINNT 5.2 mozilla-central test debug mochitests-1/5 on 2009/10/27 10:40:04

Comment 16

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256679768.1256681554.2285.gz

Comment 17

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256743930.1256745348.11725.gz

Comment 18

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256767246.1256769825.2292.gz
Linux mozilla-central test debug mochitests-1/5 on 2009/10/28 15:00:46

Comment 19

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256847846.1256849552.18393.gz
Linux mozilla-central test debug mochitests-1/5 on 2009/10/29 13:24:06

Comment 20

[Nochum Sossonko [:Natch]]

Attachment: patch

This should do it.

1) Register listeners before the "unload" event may have happened.

2) Set the timeout after we already have the load event, in case the timeout comes too soon.

Comment 21

[Mats Palmgren (inactive)]

Comment on attachment 409271 [details] [diff] [review]
patch

r=mats

Comment 22

[Nochum Sossonko [:Natch]]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Unittest/1256926080.1256927517.24632.gz&fulltext=1
Linux mozilla-central test debug mochitests-1/5 on 2009/10/30 11:08:00

Comment 23

[Phil Ringnalda (:philor)]

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

I'll let you decide when and whether it's fixed ;)

Comment 24

[Nochum Sossonko [:Natch]]

How about we just mark it fixed for now, and if there are more reports, reopen.