599623 - multiple prompt of onbeforeunload popup when close group, and Tabbrowser is *completely messed up*

Status: RESOLVED FIXED

(Firefox Graveyard :: Panorama, defect, P1)

Description

[Alice0775 White]

Build Identifier: http://hg.mozilla.org/mozilla-central/rev/71e8b5aee972
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b7pre) Gecko/20100925 Firefox/4.0b7pre ID:20100925041313

If there are tab which is include onbeforeunload(such as Google Document),
After perform close group and undo in Panorama,
Multiple prompt of onbeforeunload will popup.

AND Tabbrowser is *completely messed up*
Tabs are lost.

Reproducible: Always

Steps to Reproduce:
1. Start Minefield with new profile
2. Open Testcase
3. Open adequate number of web pages in new tabs.
4. Go Panorama
5. Divide tabs into two groups
6. Close a group which include the testcase
7. Select a remaining group
8. When popup confirmation prompts, you chose "Leave"

Actual Results:
 The confirmation prompt will appear at multiple times
 And Browser will close. the remaining group tabs are lost.

Expected Results:
 The confirmation prompt should be one,
 Browser should not close. and the remaining tab should be there.

Comment 1

[Alice0775 White]

Please read it as follows 
>After perform close group and undo in Panorama,
After perform close group in Panorama,

Comment 2

[Alice0775 White]

Attachment: Testcase

Comment 3

[Ian Gilman [:iangilman]]

The only difference between this and bug 599626 is that you hit "leave" rather than "cancel"?

Comment 4

[Alice0775 White]

(In reply to comment #3)
> The only difference between this and bug 599626 is that you hit "leave" rather
> than "cancel"?

Maybe same bug, but different behavior.

Comment 5

[Raymond Lee [:raymondlee]]

(In reply to comment #4)
> (In reply to comment #3)
> > The only difference between this and bug 599626 is that you hit "leave" rather
> > than "cancel"?
> 
> Maybe same bug, but different behavior.

The patch in bug 599626 should also fix this.

Comment 6

[Ian Gilman [:iangilman]]

Dietrich, this will be fixed by bug 599626; can you make that blocking as well?

Comment 7

[Ian Gilman [:iangilman]]

Bug 599626 has landed, so this is fixed now.