462541 - browser_339445.js crashes on my computer

Status: RESOLVED WORKSFORME

(Firefox :: Session Restore, defect)

Description

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1b2pre) Gecko/20081031 Minefield/3.1b2pre] (home, optim default) (W2Ksp4)

Comment 1

[Serge Gautherie (:sgautherie)]

The crash is triggered by enabling the following "line":
|tab.linkedBrowser.addEventListener(...);|
even when removing all the code inside
and with/without keeping the |finish();| inside.

Comment 2

[Serge Gautherie (:sgautherie)]

Nit:
|let testURL = ...| is missing a ';'.

Comment 3

[Simon Bünzli]

* Does the crash happen when using a nightly build instead of your own build?
* Do you get a crash report/stack so that you can tell us where exactly the crash happens?

NB: Our own unit test machines don't crash...

Comment 4

[Serge Gautherie (:sgautherie)]

(In reply to comment #3)
> * Does the crash happen when using a nightly build instead of your own build?

I rebuilded (with a few newer checkins), and it doesn't crash anymore...

R.WorksForMe

> * Do you get a crash report/stack so that you can tell us where exactly the
> crash happens?

No, no Breakpad nor DrWatson (or I didn't wait long enough): plain crash.

> NB: Our own unit test machines don't crash...

I know.

Comment 5

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) Nit: missing ';' [Checkin: Comment 7]

Comment 6

[Simon Bünzli]

Comment on attachment 345807 [details] [diff] [review]
(Av1) Nit: missing ';'
[Checkin: Comment 7]

Thanks.

Comment 7

[Serge Gautherie (:sgautherie)]

Comment on attachment 345807 [details] [diff] [review]
(Av1) Nit: missing ';'
[Checkin: Comment 7]

http://hg.mozilla.org/mozilla-central/rev/7e750c55fd89