1791469 - Crash in [@ nsWindow::WaylandPopupMovePlain]

Status: RESOLVED FIXED

(Core :: Widget: Gtk, defect, P3)

Description

[Bob Clary [:bc] (inactive)]

Crash report: https://crash-stats.mozilla.org/report/index/b45dfa43-1fd0-46a0-ad57-ccd680220919

MOZ_CRASH Reason: MOZ_DIAGNOSTIC_ASSERT(gtk_window_get_type_hint(((((GtkWindow*) g_type_check_instance_cast ((GTypeInstance*) ((mShell)), ((gtk_window_get_type ()))))))) == GDK_WINDOW_TYPE_HINT_UTILITY)

Top 10 frames of crashing thread:

0 libxul.so nsWindow::WaylandPopupMovePlain widget/gtk/nsWindow.cpp:2569
1 libxul.so nsWindow::WaylandPopupMoveImpl widget/gtk/nsWindow.cpp:2621
2 libxul.so nsWindow::UpdateWaylandPopupHierarchy widget/gtk/nsWindow.cpp:1887
3 libxul.so nsWindow::NativeShow widget/gtk/nsWindow.cpp:6474
4 libxul.so nsWindow::Show widget/gtk/nsWindow.cpp:940
5 libxul.so nsView::DoResetWidgetBounds view/nsView.cpp
5 libxul.so nsView::ResetWidgetBounds view/nsView.cpp:189
5 libxul.so nsViewManager::ProcessPendingUpdatesForView view/nsViewManager.cpp:355
6 libxul.so nsViewManager::ProcessPendingUpdates view/nsViewManager.cpp:942
7 libxul.so nsRefreshDriver::Tick layout/base/nsRefreshDriver.cpp:2758

This and the other crashes I experienced are on Fedora 36 and had widget.wayland.use-move-to-rect = false and occurred almost everywhere but especially attempting to view a crash report. Setting the pref to true stopped the crashes (for now).

Comment 1

[Bob Clary [:bc] (inactive)]

cc and ni ftw.

Comment 2

[Martin Stránský [:stransky] (ni? me)]

Can you test with widget.wayland.use-move-to-rect = true?
widget.wayland.use-move-to-rect is a hack for testing only.

Comment 3

[Martin Stránský [:stransky] (ni? me)]

Also please attach your about:support.
Thanks.

Comment 4

[Bob Clary [:bc] (inactive)]

(In reply to Martin Stránský [:stransky] (ni? me) from comment #2)

Can you test with widget.wayland.use-move-to-rect = true?
widget.wayland.use-move-to-rect is a hack for testing only.

As I mentioned in comment 0, widget.wayland.use-move-to-rect true does not crash.

Comment 5

[Bob Clary [:bc] (inactive)]

Attachment: bc about:support

Note this about:support is using my current setting widget.wayland.use-move-to-rect: true but the crashes occurred with it set to false.

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1789581

Comment 7

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1791469 [Wayland] Accept plain move for tooltips r?emilio

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6f90566c2ed3
[Wayland] Accept plain move for tooltips r=emilio

Comment 9

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/6f90566c2ed3