Skip to content

Commit 3297fed

Browse files
committed
Bug 1883476 - Use ShellService when setting default from the Default Agent intervention. r=nalexander
Differential Revision: https://phabricator.services.mozilla.com/D203504
1 parent 64e6419 commit 3297fed

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

toolkit/mozapps/defaultagent/BackgroundTask_defaultagent.sys.mjs

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -19,13 +19,11 @@ ChromeUtils.defineESModuleGetters(lazy, {
1919
setTimeout: "resource://gre/modules/Timer.sys.mjs",
2020
BackgroundTasksUtils: "resource://gre/modules/BackgroundTasksUtils.sys.mjs",
2121
NimbusFeatures: "resource://nimbus/ExperimentAPI.sys.mjs",
22+
// eslint-disable-next-line mozilla/no-browser-refs-in-toolkit
23+
ShellService: "resource:///modules/ShellService.sys.mjs",
2224
});
2325
XPCOMUtils.defineLazyServiceGetters(lazy, {
2426
AlertsService: ["@mozilla.org/alerts-service;1", "nsIAlertsService"],
25-
XreDirProvider: [
26-
"@mozilla.org/xre/directory-provider;1",
27-
"nsIXREDirProvider",
28-
],
2927
});
3028
ChromeUtils.defineLazyGetter(lazy, "log", () => {
3129
let { ConsoleAPI } = ChromeUtils.importESModule(
@@ -244,9 +242,9 @@ async function doTask(defaultAgent, force) {
244242
kNotificationAction.makeFirefoxDefaultButton ||
245243
notificationTelemetry.action == kNotificationAction.toastClicked
246244
) {
247-
let aumid = lazy.XreDirProvider.getInstallHash();
248-
lazy.log.info(`Setting default browser with AUMID: ${aumid}`);
249-
defaultAgent.setDefaultBrowserUserChoice(aumid, []);
245+
await lazy.ShellService.setDefaultBrowser(false).catch(e => {
246+
lazy.log.error(`setDefaultBrowser failed: ${e}`);
247+
});
250248
}
251249

252250
defaultAgent.sendPing(

0 commit comments

Comments
 (0)