1807906 - Extension devtools window shows system application menu on right-click which covers native context menu

Status: NEW

(DevTools :: about:debugging, defect, P3)

Description

[Kestrel]

Attachment: GNOME application menu covering native context menu

STR:

1. Visit about:debugging > "This Nightly" on Ubuntu 22.10 (Wayland).
2. Click the "Inspect" button for the "System theme" extension.
3. Right-click anywhere in the Developer Tools window that opens.

The native context menu appears initially but is covered with the system application menu on mouseup.

Workaround is to keeping holding down the right mouse button while choosing a context menu item or click the "Disable always on top" button in the top-right of the toolbar.

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=2f3853320ba4779f25b223fdbadb93751c838403&tochange=45761fca51fcb3d398e215675fb732fc17c00f54

Regressed by Bug 1780912.

Comment 1

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

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

:ochameau, since you are the author of the regressor, bug 1780912, could you take a look? Also, could you set the severity field?

For more information, please visit auto_nag documentation.

Comment 2

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

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

Comment 3

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

It's called from here:

https://searchfox.org/mozilla-central/rev/daf613efc5c358f3a94961d73b90472c00703838/widget/gtk/nsWindow.cpp#4754

  // right menu click on linux should also pop up a context menu
  if (StaticPrefs::ui_context_menus_after_mouseup() &&
      eventStatus.mApzStatus != nsEventStatus_eConsumeNoDefault) {
    DispatchContextMenuEventFromMouseEvent(domButton, aEvent, refPoint);
  }

so I suspect eventStatus.mApzStatus isn't nsEventStatus_eConsumeNoDefault and we may consider it as not handled then?

Comment 4

[Alexandre Poirot [:ochameau]]

Bug 1780912 made devtools window "always on top".
But this is known to be not really functional on linux because of wayland. Wayland doesn't allow the application to define "always on top" windows. Only the user can via the context menu (which you can see in comment 0).

We might disable this feature on linux because of all the troubles of wayland.
That, until wayland finally expose an API to control this.

Comment 5

[Julian Descottes [:jdescottes]]

As mentioned by Alex, we most likely should disable this on linux wayland if it's broken.

Kestrel, can you check again and confirm this is still an issue today?
Thanks

Comment 6

[Kestrel]

It is still reproducible on the latest Nightly 128.0a1 (2024-05-22) on Ubuntu 24.04 (Wayland) with the devtools window set to always on top.