[Linux] Don't use nsGConfService in nsGNOMEShellService.cpp

RESOLVED FIXED in Firefox 68

Status

()

enhancement
RESOLVED FIXED
3 months ago
2 months ago

People

(Reporter: stransky, Assigned: stransky)

Tracking

Trunk
mozilla68
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox67 wontfix, firefox68 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

3 months ago

gconf is deprecated and should not be used if GSettings is available.

Status: NEW → RESOLVED
Last Resolved: 3 months ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1433685
Assignee

Comment 3

2 months ago

Let's handle the nsGNOMEShellService.cpp part here.

Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Summary: [Linux] Don't use nsGConfService to get/set default browser → [Linux] Don't use nsGConfService in nsGNOMEShellService.cpp
Assignee

Updated

2 months ago
Status: REOPENED → NEW
Assignee

Updated

2 months ago
Blocks: 1433685
Assignee

Updated

2 months ago
Assignee: nobody → stransky
Assignee

Comment 4

2 months ago

Don't use gconf service to get/set default browser as it breaks recent Gnome environment.
Also use XDG_CURRENT_DESKTOP to check actual desktop.

Assignee

Updated

2 months ago
Keywords: checkin-needed

Comment 5

2 months ago

Pushed by nerli@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/017eefc950b0
[Linux] Don't use nsGConfService in nsGNOMEShellService.cpp, r=glandium

Keywords: checkin-needed

Comment 6

2 months ago
bugherder
Status: NEW → RESOLVED
Last Resolved: 3 months ago2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.