So I see bug 337036 almost half the time when closing windows. Simple testcase:
javascript:alert('aaa')
Then click "ok".
Half the time I crash (inside GTK); that's bug 337036 (which has a slightly different stack from this one). The other half, I get:
(Gecko:12931): GLib-GObject-WARNING **: invalid cast from `(null)' to `GtkWidget'
(Gecko:12931): GLib-GObject-WARNING **: invalid cast from `(null)' to `GObject'
(Gecko:12931): GLib-GObject-CRITICAL **: file gobject.c: line 1642 (g_object_get_data): assertion `G_IS_OBJECT (object)' failed
When I crash, mIMEData is false. So I wouldn't even hit the codepath the patch in this bug is changing. Furthermore, in IM_get_owning_window I have a non-null aArea and a non-null aArea->inner_window. But the call to get_gtk_widget_for_gdk_window crashes.
So as far as I can tell, either bug 337036 is a different bug or this patch is wallpaper that doesn't fix the underlying issue.
So I see bug 337036 almost half the time when closing windows. Simple testcase:
javascript: alert(' aaa')
Then click "ok".
Half the time I crash (inside GTK); that's bug 337036 (which has a slightly different stack from this one). The other half, I get:
(Gecko:12931): GLib-GObject- WARNING **: invalid cast from `(null)' to `GtkWidget' WARNING **: invalid cast from `(null)' to `GObject' CRITICAL **: file gobject.c: line 1642 (g_object_ get_data) : assertion `G_IS_OBJECT (object)' failed
(Gecko:12931): GLib-GObject-
(Gecko:12931): GLib-GObject-
When I crash, mIMEData is false. So I wouldn't even hit the codepath the patch in this bug is changing. Furthermore, in IM_get_ owning_ window I have a non-null aArea and a non-null aArea-> inner_window. But the call to get_gtk_ widget_ for_gdk_ window crashes.
So as far as I can tell, either bug 337036 is a different bug or this patch is wallpaper that doesn't fix the underlying issue.