728541 - [SeaMonkey] "TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/layout/xul/test/browser_bug706743.js | an unexpected uncaught JS exception reported through window.onerror - [...] browser_bug703210.js:50"

Status: VERIFIED FIXED

(Core :: XUL, defect, P2)

Description

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1329539399.1329544677.8733.gz
OS X 10.6 comm-central-trunk debug test mochitest-other on 2012/02/17 20:29:59
+
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey-Aurora/1329548148.1329554173.27000.gz
WINNT 5.2 comm-aurora debug test mochitest-other on 2012/02/17 22:55:48
+
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey-Beta/1329553178.1329556949.32205.gz
Linux comm-beta debug test mochitest-other on 2012/02/18 00:19:38
{
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/layout/xul/test/browser_bug706743.js | an unexpected uncaught JS exception reported through window.onerror - EventUtils.synthesizeMouseAtCenter is not a function at chrome://mochitests/content/browser/layout/xul/test/browser_bug703210.js:50
}

browser_bug703210.js is leaking into (the next test) browser_bug706743.js!

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) browser_bug703210.js: Add a missing removeEventListener() [Checked in: Comments 3 and 6]

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Oh, thank you. And sorry for the mistake.

Comment 3

[Serge Gautherie (:sgautherie)]

Comment on attachment 598512 [details] [diff] [review]
(Av1) browser_bug703210.js: Add a missing removeEventListener()
[Checked in: Comments 3 and 6]

https://hg.mozilla.org/mozilla-central/rev/e06ab2cf18b8


[Approval Request Comment]
Regression caused by (bug #): Bug 703210.
User impact if declined: None, but "false" perma-orange on SeaMonkey.
Testing completed (on m-c, etc.): locally and this comment.
Risk to taking this patch (and alternatives if risky): None, test only.
String changes made by this patch: None.

Comment 4

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1329624302.1329629638.17173.gz
OS X 10.6 comm-central-trunk debug test mochitest-other on 2012/02/18 20:05:02

V.Fixed

Comment 5

[Alex Keybl [:akeybl]]

Comment on attachment 598512 [details] [diff] [review]
(Av1) browser_bug703210.js: Add a missing removeEventListener()
[Checked in: Comments 3 and 6]

[Triage Comment]
Test only change - approved for Aurora 12 and Beta 11.

Comment 6

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 598512 [details] [diff] [review]
(Av1) browser_bug703210.js: Add a missing removeEventListener()
[Checked in: Comments 3 and 6]

http://hg.mozilla.org/releases/mozilla-aurora/rev/026e5d8f9638
http://hg.mozilla.org/releases/mozilla-beta/rev/61192dc04957

Comment 7

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey-Aurora/1329891487.1329895867.22195.gz
OS X 10.6 comm-aurora debug test mochitest-other on 2012/02/21 22:18:07

firefox12: verified.

***

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey-Beta/1329894953.1329898771.26982.gz
WINNT 5.2 comm-beta debug test mochitest-other on 2012/02/21 23:15:53

firefox11: verified.