Closed Bug 1481392 Opened 2 years ago Closed 2 years ago

[flatpak] nsFlatpakPrintPortal: Make sure we don't have any DBus connection open after print

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: stransky, Assigned: ozoder)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

At widget/gtk/nsPrintDialogGTK.cpp, nsFlatpakPrintPortal implementation:

In context of Bug 1480517 we need to make sure we don't have any active DBus connection pending. We may g_dbus_connection_signal_unsubscribe() the signal handler for instance.
Ondrej, can you please look at it? Thanks.
Assignee: jhorak → ozoder
Comment on attachment 9002694 [details]
Bug 1481392 - Close DBus connection after print, r?stransky

Martin Stránský [:stransky] has approved the revision.
Attachment #9002694 - Flags: review+
Keywords: checkin-needed
Pushed by ebalazs@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/624a0fcb6618
Close DBus connection after print, r=stransky
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/624a0fcb6618
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.