From c96af331fc71fd212142aeef6ceb87f0d3c6e110 Mon Sep 17 00:00:00 2001 From: Mark Banner Date: Tue, 29 Apr 2025 19:57:01 +0000 Subject: [PATCH] Bug 1963014 - Fix JSDoc Issues in ResetProfile.sys.mjs. r=firefox-desktop-core-reviewers ,frontend-codestyle-reviewers,mossop Differential Revision: https://phabricator.services.mozilla.com/D246914 --- .eslintrc-rollouts.js | 4 ++-- toolkit/modules/ResetProfile.sys.mjs | 5 ++++- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/.eslintrc-rollouts.js b/.eslintrc-rollouts.js index f0a8ad56c1f9..98231a3dd0c2 100644 --- a/.eslintrc-rollouts.js +++ b/.eslintrc-rollouts.js @@ -391,7 +391,7 @@ const rollouts = [ "toolkit/components/workerloader/require.js", "toolkit/content/**", "toolkit/crashreporter/**", - "toolkit/modules/{C,Da,E10SUtils,F,G,In,J,Ke,L,N,P,R,S,Up,W}*.sys.mjs", + "toolkit/modules/{C,Da,E10SUtils,F,G,In,J,Ke,L,N,P,Rem,S,Up,W}*.sys.mjs", "toolkit/modules/sessionstore/**", "toolkit/modules/subprocess/**", "toolkit/modules/tests/**", @@ -555,7 +555,7 @@ const rollouts = [ "toolkit/components/workerloader/require.js", "toolkit/content/**", "toolkit/crashreporter/**", - "toolkit/modules/{Asy,B,C,Da,E10SUtils,F,G,In,JS,Ke,L,Ne,P,R,S,Up,W}*.sys.mjs", + "toolkit/modules/{Asy,B,C,Da,E10SUtils,F,G,In,JS,Ke,L,Ne,P,Rem,S,Up,W}*.sys.mjs", "toolkit/modules/sessionstore/**", "toolkit/modules/subprocess/**", "toolkit/modules/tests/**", diff --git a/toolkit/modules/ResetProfile.sys.mjs b/toolkit/modules/ResetProfile.sys.mjs index f3575f3ce42d..e1b8ae608185 100644 --- a/toolkit/modules/ResetProfile.sys.mjs +++ b/toolkit/modules/ResetProfile.sys.mjs @@ -14,6 +14,7 @@ ChromeUtils.defineLazyGetter(lazy, "MigrationUtils", () => { try { let { MigrationUtils } = ChromeUtils.importESModule( + // eslint-disable-next-line mozilla/no-browser-refs-in-toolkit "resource:///modules/MigrationUtils.sys.mjs" ); return MigrationUtils; @@ -29,7 +30,7 @@ export var ResetProfile = { /** * Check if reset is supported for the currently running profile. * - * @return boolean whether reset is supported. + * @returns {boolean} whether reset is supported. */ resetSupported() { if (Services.policies && !Services.policies.isAllowed("profileRefresh")) { @@ -66,6 +67,8 @@ export var ResetProfile = { /** * Ask the user if they wish to restart the application to reset the profile. + * + * @param {Window} window */ async openConfirmationDialog(window) { let win = window;