[wayland] XDG annotation is not freed with native menubar enabled
Categories
(Core :: Widget: Gtk, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr115 | --- | unaffected |
firefox123 | --- | unaffected |
firefox124 | --- | unaffected |
firefox125 | - | disabled |
firefox126 | --- | disabled |
People
(Reporter: emilio, Assigned: emilio)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: memory-leak, regression)
Attachments
(2 files)
Bug 1883184 introduced a call to xdg_dbus_annotation_manager_v1_create_surface, but nothing calls the equivalent _delete
.
Comment 1•11 months ago
|
||
:uhhadd, since you are the author of the regressor, bug 1883184, could you take a look?
For more information, please visit BugBot documentation.
Comment 2•11 months ago
|
||
Set release status flags based on info from the regressing bug 1883184
Updated•11 months ago
|
Comment 3•11 months ago
|
||
The bug is marked as tracked for firefox125 (nightly). We have limited time to fix this, the soft freeze is in 7 days. However, the bug still isn't assigned.
:gcp, could you please find an assignee for this tracked bug? Given that it is a regression and we know the cause, we could also simply backout the regressor. If you disagree with the tracking decision, please talk with the release managers.
For more information, please visit BugBot documentation.
Updated•11 months ago
|
Comment 4•11 months ago
|
||
Emilio, the regressing bug says the feature is disabled by default. So is this leak an actual issue? If it is, we probably need to back it out?
Assignee | ||
Comment 5•11 months ago
|
||
Updated•11 months ago
|
Assignee | ||
Comment 6•11 months ago
|
||
Until it has a final shape, at least, since there's ongoing work and
discussion in the MR.
Assignee | ||
Comment 7•11 months ago
|
||
Yes, it's not enabled by default.
Assignee | ||
Updated•11 months ago
|
Updated•11 months ago
|
Comment 9•11 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/655c9f682b98
https://hg.mozilla.org/mozilla-central/rev/1c5001e4717c
Comment 10•11 months ago
|
||
Set release status flags based on info from the regressing bug 1883184
Updated•11 months ago
|
Description
•