76367 - hang opening new browser from mail compose

Status: RESOLVED FIXED

(Core :: XBL, defect, P1)

Description

[Chris Waterson]

SUMMARY

Mozilla hangs in an infinite loop when opening a new browser window from the
mail compose window.

To Reproduce:

1. Start mozilla.
2. Open a mail compose window (e.g., by pressing Ctrl+M)
3. Open a new browser window by pression Ctrl+N in the mail compose window

Expected:

Browser window opens.

Action:

App hangs, CPU pegged.

Comment 1

[Chris Waterson]

Attachment: stack trace at hang; infinite loop in nsXBLWindowHandler::WalkHandlersInternal

Comment 2

[Chris Waterson]

Attachment: stack trace; hang at nsXBLWindowHandler::WalkHandlersInternal

Comment 3

[Chris Waterson]

Aw crap, ignore the first stack trace. I fat-fingered emacs. The second stack
trace has the goods; specifically, hanging in
nsXBLWindowHandler::WalkHandlersInternal().

Comment 4

[doctor__j]

However, when I go to File Menu -> New -> Navigator Window, I cannot bring up a
new navigator window.  Another bug???

Comment 5

[doctor__j]

Is this a dup of Bug 75517 ??

Comment 6

[Asa Dotzler [:asa]]

*** Bug 75517 has been marked as a duplicate of this bug. ***

Comment 7

[Peter Trudelle]

targetting moz0.9/p1

Comment 8

[David Hyatt]

Attachment: Oops. Continue was not the right thing to do here.

Comment 9

[David Hyatt]

Attachment: Also adding menu assertion to help XUL authors.

Comment 10

[Chris Waterson]

sr=waterson. thanks for the speedy service!

Comment 11

[Asa Dotzler [:asa]]

a= [e-mail] for checkin to 0.9

Does this still need a r= or is it ready to go?

Comment 12

[David Hyatt]

Fixed the hang and added assertions.  Note that the mailcompose window is 
missing the <command> element for new navigator window, and so a separate bug 
should be filed against the mailcompose window for that issue.  

With my patch CTRL+N will no longer hang, but like the menu item, it won't work.

Comment 13

[Brian Kaul]

*** Bug 76764 has been marked as a duplicate of this bug. ***

Comment 14

[HJ]

David that missing <command> issue has been fixed by me in bug 76764.