1980359 - WaylandMonitor / ScreenGetter leaks

Status: RESOLVED FIXED

(Core :: Widget: Gtk, defect)

Description

[Emilio Cobos Álvarez [:emilio]]

Seems like a regression from bug 1971161.

Running any mochitest locally:

$ mach mochitest browser/components/customizableui/test/browser_947914_button_zoomReset.js

Ends with:

 0:26.17 INFO leakcheck: default leaked 2 Screen
 0:26.17 INFO leakcheck: default leaked 1 ScreenGetterGtk
 0:26.17 INFO leakcheck: default leaked 1 StringBuffer
 0:26.17 INFO leakcheck: default leaked 4 WaylandMonitor

Comment 1

[Emilio Cobos Álvarez [:emilio]]

The StringBuffer is unrelated (I tracked it down to the global DRM device).

Comment 2

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

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

Comment 4

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

Attachment: Bug 1980359 [Wayland] Finish WaylandMonitor / ScreenGetterGtk early r?emilio

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/2fdb52603f91
https://hg.mozilla.org/integration/autoland/rev/2c1d4a94f8b5
[Wayland] Finish WaylandMonitor / ScreenGetterGtk early r=emilio

Comment 6

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/2c1d4a94f8b5

Comment 7

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

The patch landed in nightly and beta is affected.
:stransky, is this bug important enough to require an uplift?

• If yes, please nominate the patch for beta approval.
  • See https://wiki.mozilla.org/Release_Management/Requesting_an_Uplift for documentation on how to request an uplift.
• If no, please set status-firefox142 to wontfix.

For more information, please visit BugBot documentation.