15672 - App exit after XP menu usage, closing dialog with Ctrl-W.

Status: VERIFIED FIXED

(Core :: XUL, defect, P2)

Description

[Syd Logan]

Steps to reproduce:

-- Start apprunner
-- Tasks->Messenger
-- Open up File menu using the mouse
-- Use Ctrl-W to dismiss the dialog
-- The messenger window background will clear but never really closes
-- You may see a rectangular region in the vicinity of the dialog that is about
the size of the File menu that was posted above.
-- Click on something in the mozilla dialog.
-- Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkWidget' will display
on the console.
-- apprunner will exit.

Comment 1

[Claudius Gayle]

I can't exactly follow your steps to reproduce, but this reminds me of bug 15308
I wonder if the stack traces are the same.

Comment 2

[Peter Trudelle]

reproduced in today's opt bits, sort of.  The Messenger window does close on the
Ctrl-W, but there is a grey area left where the menu was.  Trying to use the
remaining Navigator window results in the warning described, and a segfault.
setting p2 for m12, since this seems like a contrived scenario.

Comment 3

[Peter Trudelle]

mass-moving all m12 bugs to m13

Comment 4

[Peter Trudelle]

Putting on beta1 radar.

Comment 5

[leger]

claudius, can you try with latest build..thanks!

Comment 6

[Claudius Gayle]

talk about contrived... with some effort I WAS able to reproduce this with the 2000012709 linux build w/ RH6.
I had a hell of a time gettig ctrl-w to work with the file menu open though, I'm not exactly sure if that was a
requirement. The mail window seemed to close properly and without incident. However, after doing so clicking
in the browser content area printing the following to the console before quitting:

Gdk-CRITICAL **: file gdkwindow.c: line 1474 (gdk_window_get_origin): assertion 'window != NULL' failed.

Gdk-CRITICAL **: file gdkwindow.c: line 1385 (gdk_window_get_size): assertion 'window != NULL' failed.

Comment 7

[Peter Trudelle]

removing from beta1 radar.

Comment 8

[leger]

Putting on PDT+ radar for beta1.  Great work claudius!
Putting back on beta1

Comment 9

[Stuart Parmenter]

This is fixed.  I'm not sure if I fixed it, hyatt fixed it or saari fixed it,
but it is fixed now.

Comment 10

[Claudius Gayle]

yup. VERIFIED with 2000021609 RH6 build