Closed Bug 1703073 Opened 3 years ago Closed 3 years ago

Firefox crash when adding bookmark while location popup is open

Categories

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

Firefox 87
defect

Tracking

()

RESOLVED FIXED
90 Branch
Tracking Status
firefox90 --- fixed

People

(Reporter: mozbug, Assigned: stransky)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:87.0) Gecko/20100101 Firefox/87.0

Steps to reproduce:

View: https://palegreendot.co.uk/collections/fresh-alcohol-fortnightly/products/premium-veg-box-fortnightly-subscription?selling_plan=441122970&variant=37737131016346

A location permission popup appears.
While this popup is still open, click the button to bookmark a page.

Actual results:

The star turns blue to confirm the bookmark, but nothing else happens. The browser completely freezes (prompting a 'Firefox is not responding' dialog), or just crashes.

@stransky I'm using Wayland, so this could be a Wayland-specific issue.

The Bugbug bot thinks this bug should belong to the 'Firefox::Bookmarks & History' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Component: Untriaged → Bookmarks & History

Hi Sam,
I was unable to reproduce this on Windows 10 but as you also mentioned this could be a Wayland specific issue.
Can you please share here the crash reports you have? To do that, please navigate to about:crashes, submit any available crashes after reproducing it and share it here.
Moving this over to the wayland specific component in the meantime. Thanks!

Component: Bookmarks & History → Widget: Gtk
Product: Firefox → Core

Forgot to NI?

Flags: needinfo?(mozbug)
Priority: -- → P1

Most recent crash report is from 14th January, seems that these crashes are not getting logged (I've had plenty of crashes recently due to OOM problems, and it looks they are not here either).

Flags: needinfo?(mozbug)

This is caused by Wayland architecture. Generally when we hide any popup in popup chain we need to hide/reposition all child popups. It also affects all popup operations at nsWindow::NativeMoveResizeWaylandPopup() where popup is hidden/shown due to https://gitlab.gnome.org/GNOME/gtk/issues/1986

Assignee: nobody → stransky
  • Keep strong popup hierarchy from both layout and widget perspective and use only
    topmost popups as parents for newly opened popups.
  • Always close popup windows with all opened child popups.
  • Always close tooltip windows when new popup is created.
  • Introduce 'WidgetPopup' log to trace popups changes.
  • Export correct window roles (Toplevel, Popup, Dialog).
Pushed by stransky@redhat.com:
https://hg.mozilla.org/integration/autoland/rev/72bdd02e4155
[Wayland] Rework Wayland popup hierarchy management, r=jhorak
Status: UNCONFIRMED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
Regressions: 1720694
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: