commit 9bccb7025a05776fb52252fe8f4f6b9a6960fbce parent ecc3c84c8a5601edd4e675823bd06bbff7a528ed Author: Mark Banner <standard8@mozilla.com> Date: Wed, 29 Oct 2025 11:19:47 +0000 Bug 1996350 - Manual fixes for enabling ESLint rule jsdoc/no-multi-asterisks everywhere. r=frontend-codestyle-reviewers,necko-reviewers,media-playback-reviewers,devtools-reviewers,sync-reviewers,firefox-desktop-core-reviewers ,firefox-svg-reviewers,mstriemer,nchevobbe,mconley,skhamis,alwu,mossop Differential Revision: https://phabricator.services.mozilla.com/D270033 Diffstat:
25 files changed, 59 insertions(+), 62 deletions(-)
diff --git a/browser/components/BrowserContentHandler.sys.mjs b/browser/components/BrowserContentHandler.sys.mjs @@ -832,7 +832,8 @@ nsBrowserContentHandler.prototype = { * or equal to the maxVersion set by the experiment, we'll try to use * the experiment override URL instead of the default or the * update-provided URL. Additional policy checks are done in - * @see getPostUpdateOverridePage */ + * @see getPostUpdateOverridePage + */ const nimbusOverrideUrl = Services.urlFormatter.formatURLPref( "startup.homepage_override_url_nimbus" ); diff --git a/devtools/client/inspector/rules/models/rule.js b/devtools/client/inspector/rules/models/rule.js @@ -503,7 +503,7 @@ class Rule { * The value to be used for the preview * @param {String} priority * The property's priority (either "important" or an empty string). - **@return {Promise} + * @return {Promise} */ previewPropertyValue(property, value, priority) { this.elementStyle.ruleView.emitForTests("start-preview-property-value"); diff --git a/devtools/client/memory/constants.js b/devtools/client/memory/constants.js @@ -14,7 +14,7 @@ exports.ALLOCATION_RECORDING_OPTIONS = { // in `devtools/client/themes/memory.css` exports.TREE_ROW_HEIGHT = 18; -/** * Actions ******************************************************************/ +/*** Actions ******************************************************************/ const actions = (exports.actions = {}); @@ -123,7 +123,7 @@ actions.RESIZE_SHORTEST_PATHS = "resize-shortest-paths"; // Fired when the memory front is changed. actions.UPDATE_MEMORY_FRONT = "update-memory-front"; -/** * Census Displays ***************************************************************/ +/*** Census Displays ***************************************************************/ const COUNT = Object.freeze({ by: "count", count: true, bytes: true }); const INTERNAL_TYPE = Object.freeze({ by: "internalType", then: COUNT }); @@ -237,7 +237,7 @@ exports.treeMapDisplays = Object.freeze({ }), }); -/** * View States **************************************************************/ +/*** View States **************************************************************/ /** * The various main views that the tool can be in. @@ -249,7 +249,7 @@ viewState.DOMINATOR_TREE = "view-state-dominator-tree"; viewState.TREE_MAP = "view-state-tree-map"; viewState.INDIVIDUALS = "view-state-individuals"; -/** * Snapshot States **********************************************************/ +/*** Snapshot States **********************************************************/ const snapshotState = (exports.snapshotState = Object.create(null)); @@ -301,7 +301,7 @@ treeMapState.SAVING = "tree-map-state-saving"; treeMapState.SAVED = "tree-map-state-saved"; treeMapState.ERROR = "tree-map-state-error"; -/** * Diffing States ***********************************************************/ +/*** Diffing States ***********************************************************/ /* * Various states the diffing model can be in. @@ -325,7 +325,7 @@ diffingState.TOOK_DIFF = "diffing-state-took-diff"; // An error occurred while computing the diff. diffingState.ERROR = "diffing-state-error"; -/** * Dominator Tree States ****************************************************/ +/*** Dominator Tree States ****************************************************/ /* * Various states the dominator tree model can be in. @@ -343,7 +343,7 @@ dominatorTreeState.INCREMENTAL_FETCHING = "dominator-tree-state-incremental-fetching"; dominatorTreeState.ERROR = "dominator-tree-state-error"; -/** * States for Individuals Model *********************************************/ +/*** States for Individuals Model *********************************************/ /* * Various states the individuals model can be in. diff --git a/devtools/shared/heapsnapshot/CensusUtils.js b/devtools/shared/heapsnapshot/CensusUtils.js @@ -7,7 +7,7 @@ const { flatten, } = require("resource://devtools/shared/ThreadSafeDevToolsUtils.js"); -/** * Visitor ****************************************************************/ +/*** Visitor ****************************************************************/ /** * A Visitor visits each node and edge of a census report tree as the census @@ -62,7 +62,7 @@ Visitor.prototype.exit = function () {}; */ Visitor.prototype.count = function () {}; -/** * getReportEdges *********************************************************/ +/*** getReportEdges *********************************************************/ const EDGES = Object.create(null); @@ -143,7 +143,7 @@ function getReportEdges(breakdown, report) { } exports.getReportEdges = getReportEdges; -/** * walk *******************************************************************/ +/*** walk *******************************************************************/ function recursiveWalk(breakdown, edge, report, visitor) { if (breakdown.by === "count") { @@ -181,7 +181,7 @@ function walk(breakdown, report, visitor) { } exports.walk = walk; -/** * diff *******************************************************************/ +/*** diff *******************************************************************/ /** * Return true if the object is a Map, false otherwise. Works with Map objects diff --git a/devtools/shared/heapsnapshot/HeapAnalysesClient.js b/devtools/shared/heapsnapshot/HeapAnalysesClient.js @@ -80,7 +80,7 @@ HeapAnalysesClient.prototype.getCreationTime = function (snapshotFilePath) { return this._worker.performTask("getCreationTime", snapshotFilePath); }; -/** * Censuses *****************************************************************/ +/*** Censuses *****************************************************************/ /** * Ask the worker to perform a census analysis on the heap snapshot with the @@ -209,7 +209,7 @@ HeapAnalysesClient.prototype.takeCensusDiff = function ( }); }; -/** * Dominator Trees **********************************************************/ +/*** Dominator Trees **********************************************************/ /** * Compute the dominator tree of the heap snapshot loaded from the given file diff --git a/devtools/shared/transport/tests/xpcshell/head_dbg.js b/devtools/shared/transport/tests/xpcshell/head_dbg.js @@ -138,7 +138,7 @@ function writeTestTempFile(fileName, content) { } } -/** * Transport Factories ***/ +/*** Transport Factories ***/ var socket_transport = async function () { if (!DevToolsServer.listeningSockets) { @@ -163,7 +163,7 @@ function local_transport() { return Promise.resolve(DevToolsServer.connectPipe()); } -/** * Sample Data ***/ +/*** Sample Data ***/ var gReallyLong; function really_long() { diff --git a/devtools/shared/transport/tests/xpcshell/test_bulk_error.js b/devtools/shared/transport/tests/xpcshell/test_bulk_error.js @@ -16,7 +16,7 @@ function run_test() { run_next_test(); } -/** * Sample Bulk Actor ***/ +/*** Sample Bulk Actor ***/ const protocol = require("resource://devtools/shared/protocol.js"); const { Arg, RetVal } = protocol; const { Actor } = require("resource://devtools/shared/protocol/Actor.js"); @@ -74,7 +74,7 @@ function add_test_bulk_actor() { ); } -/** * Tests ***/ +/*** Tests ***/ var test_string_error = async function (transportFactory) { const transport = await transportFactory(); diff --git a/devtools/shared/transport/tests/xpcshell/test_client_server_bulk.js b/devtools/shared/transport/tests/xpcshell/test_client_server_bulk.js @@ -31,7 +31,7 @@ add_task(async function () { DevToolsServer.destroy(); }); -/** * Sample Bulk Actor ***/ +/*** Sample Bulk Actor ***/ const protocol = require("resource://devtools/shared/protocol.js"); const { Arg, RetVal } = protocol; const { Actor } = require("resource://devtools/shared/protocol/Actor.js"); @@ -144,7 +144,7 @@ function add_test_bulk_actor() { ); } -/** * Reply Handlers ***/ +/*** Reply Handlers ***/ var replyHandlers = { async json(reply) { @@ -168,7 +168,7 @@ var replyHandlers = { }, }; -/** * Tests ***/ +/*** Tests ***/ var test_bulk_request_cs = async function ( transportFactory, @@ -270,7 +270,7 @@ var test_json_request_cs = async function ( return serverDeferred; }; -/** * Test Utils ***/ +/*** Test Utils ***/ function verify_files() { const reallyLong = really_long(); diff --git a/devtools/shared/transport/tests/xpcshell/test_delimited_read.js b/devtools/shared/transport/tests/xpcshell/test_delimited_read.js @@ -21,7 +21,7 @@ function run_test() { run_next_test(); } -/** * Tests ***/ +/*** Tests ***/ function test_delimited_read(input, expected) { input = new StringInputStream(input); diff --git a/devtools/shared/transport/tests/xpcshell/test_queue.js b/devtools/shared/transport/tests/xpcshell/test_queue.js @@ -24,7 +24,7 @@ function run_test() { run_next_test(); } -/** * Tests ***/ +/*** Tests ***/ var test_transport = async function (transportFactory) { let clientResolve; @@ -153,7 +153,7 @@ var test_transport = async function (transportFactory) { return Promise.all([clientDeferred, serverDeferred]); }; -/** * Test Utils ***/ +/*** Test Utils ***/ function verify() { const reallyLong = really_long(); diff --git a/devtools/shared/transport/tests/xpcshell/test_transport_bulk.js b/devtools/shared/transport/tests/xpcshell/test_transport_bulk.js @@ -19,7 +19,7 @@ function run_test() { run_next_test(); } -/** * Tests ***/ +/*** Tests ***/ /** * This tests a one-way bulk transfer at the transport layer. @@ -124,7 +124,7 @@ var test_bulk_transfer_transport = async function (transportFactory) { return Promise.all([clientDeferred, serverDeferred]); }; -/** * Test Utils ***/ +/*** Test Utils ***/ function verify() { const reallyLong = really_long(); diff --git a/dom/base/test/chrome/test_bug1339722.html b/dom/base/test/chrome/test_bug1339722.html @@ -22,7 +22,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=1339722 * 3. In the event handler, delete the iframe so that the frameloader would be * destroyed in the middle of ReallyStartLoading. * 4. Verify that it doesn't crash. - **/ + */ // This topic used to be http-on-useragent-request, but that got removed in // bug 1513574. on-modify-request is called around the same time, and should diff --git a/dom/base/test/test_bug1022229.html b/dom/base/test/test_bug1022229.html @@ -12,7 +12,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=1022229 /** * Test for postMessage between sandboxed iframe and non-sandboxed window. - **/ + */ SimpleTest.waitForExplicitFinish(); function go() { var ifr = document.createElement('iframe'); diff --git a/dom/manifest/test/test_ImageObjectProcessor_purpose.html b/dom/manifest/test/test_ImageObjectProcessor_purpose.html @@ -13,7 +13,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id= /** * Image object's purpose member * https://w3c.github.io/manifest/#purpose-member - **/ + */ "use strict"; const testManifest = { diff --git a/dom/manifest/test/test_ManifestProcessor_id.html b/dom/manifest/test/test_ManifestProcessor_id.html @@ -13,7 +13,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=1731940 /** * Manifest id member * https://w3c.github.io/manifest/#id-member - **/ + */ for (const type of typeTests) { data.jsonText = JSON.stringify({ id: type, diff --git a/dom/media/test/test_video_in_audio_element.html b/dom/media/test/test_video_in_audio_element.html @@ -13,7 +13,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=1060896 /** * Test for Bug 1060896; tests that loading a video inside an audio element works. - **/ + */ var manager = new MediaTestManager; diff --git a/dom/svg/test/test_SVGxxxList.xhtml b/dom/svg/test/test_SVGxxxList.xhtml @@ -1064,7 +1064,7 @@ function run_animation_timeline_tests() { t.old_animVal_items = get_array_of_list_items(t.animVal); - /** ****************** t = 1s ********************/ + /******************** t = 1s ********************/ svg.setCurrentTime(1); // begin first animation @@ -1107,7 +1107,7 @@ function run_animation_timeline_tests() { t.old_animVal_items = get_array_of_list_items(t.animVal); - /** ****************** t = 2s ********************/ + /******************** t = 2s ********************/ svg.setCurrentTime(2); // begin override animation @@ -1130,7 +1130,7 @@ function run_animation_timeline_tests() { t.old_animVal_items = get_array_of_list_items(t.animVal); - /** ****************** t = 3s ********************/ + /******************** t = 3s ********************/ svg.setCurrentTime(3); // end of override animation @@ -1153,7 +1153,7 @@ function run_animation_timeline_tests() { t.old_animVal_items = get_array_of_list_items(t.animVal); - /** ****************** t = 5s ********************/ + /******************** t = 5s ********************/ svg.setCurrentTime(5); // animation repeat point @@ -1169,7 +1169,7 @@ function run_animation_timeline_tests() { "same objects as were at that index before the repeat occurred."); - /** ****************** t = 6s ********************/ + /******************** t = 6s ********************/ svg.setCurrentTime(6); // inside animation repeat @@ -1185,7 +1185,7 @@ function run_animation_timeline_tests() { "same objects as were at that index before the repeat occurred."); - /** ****************** t = 7s ********************/ + /******************** t = 7s ********************/ svg.setCurrentTime(7); // start of additive="sum" animation @@ -1229,7 +1229,7 @@ function run_animation_timeline_tests() { // SVGLengthListSMILAttr::Add(). - /** ****************** t = 13s ********************/ + /******************** t = 13s ********************/ svg.setCurrentTime(13); // all animations have finished, but one is frozen diff --git a/dom/tests/mochitest/pointerlock/file_retargetMouseEvents.html b/dom/tests/mochitest/pointerlock/file_retargetMouseEvents.html @@ -67,7 +67,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=633602 * Mouse events will only happen when the pointer is locked * and if the pointer is locked all the mouse events should be * retargetted to the locked element - **/ + */ var childMoveTest = function() { clearInterval(mouseMoveIntervalID); childStats.mouseMove = true; diff --git a/dom/tests/mochitest/pointerlock/test_pointerlock-api.html b/dom/tests/mochitest/pointerlock/test_pointerlock-api.html @@ -27,7 +27,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=633602 * mochitests' iframe doesn't have an allowfullscreen attribute. To get * around this, all tests are run in a child window, which can go fullscreen. * This method is borrowed from dom/html/test/test_fullscreen-api.html. - **/ + */ SimpleTest.waitForExplicitFinish(); diff --git a/intl/uconv/tests/test_bug959058-1.html b/intl/uconv/tests/test_bug959058-1.html @@ -10,7 +10,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=959058 <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css"/> <script type="application/javascript"> - /** Test for Bug 959058 **/ + /** Test for Bug 959058 */ is("”9¸2", "\uD83C\uDF54", "Should have gotten a hamburger."); diff --git a/intl/uconv/tests/test_bug959058-2.html b/intl/uconv/tests/test_bug959058-2.html @@ -10,7 +10,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=959058 <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css"/> <script type="application/javascript"> - /** Test for Bug 959058 **/ + /** Test for Bug 959058 */ is("€", "\u20AC", "Should have gotten euro."); diff --git a/netwerk/test/httpserver/test/test_async_response_sending.js b/netwerk/test/httpserver/test/test_async_response_sending.js @@ -72,11 +72,9 @@ const LATTER_HALF_THIRD_SEGMENT = [11, 12]; const TWO_HALF_SEGMENTS = [1, 2, 1, 2]; -/** - * ******* +/********* * TESTS * - ******** - */ + *********/ var tests = [ sourceClosedWithoutWrite, diff --git a/services/sync/modules/engines/extension-storage.sys.mjs b/services/sync/modules/engines/extension-storage.sys.mjs @@ -134,13 +134,11 @@ Object.setPrototypeOf( BridgedEngine.prototype ); -/** - ***************************************************************************** +/******************************************************************************* * * Deprecated support for Kinto * - ***************************************************************************** - */ + ******************************************************************************/ /** * The Engine that manages syncing for the web extension "storage" diff --git a/toolkit/content/widgets/lit-utils.mjs b/toolkit/content/widgets/lit-utils.mjs @@ -31,7 +31,7 @@ function queryAll(el, selector) { /** * MozLitElement provides extensions to the lit-provided LitElement class. * - ******* + * --------- * * `@query` support (define a getter for a querySelector): * @@ -52,14 +52,14 @@ function queryAll(el, selector) { * get anotherName() { * return this.renderRoot?.querySelectorAll(".selectorFor .querySelectorAll"); * } - ******* + * --------- * * Automatic Fluent support for shadow DOM. * * Fluent requires that a shadowRoot be connected before it can use Fluent. * Shadow roots will get connected automatically. * - ******* + * --------- * * Automatic Fluent support for localized Reactive Properties * @@ -67,7 +67,7 @@ function queryAll(el, selector) { * property definition and it will automatically be added to the data-l10n-attrs * attribute so that fluent will allow setting the attribute. * - ******* + * --------- * * Mapped properties support (moving a standard attribute to rendered content) * @@ -77,7 +77,7 @@ function queryAll(el, selector) { * definition and the attribute will be removed from the host when it is set. * Note that the attribute can not be unset once it is set. * - ******* + * --------- * * Test helper for sending events after a change: `dispatchOnUpdateComplete` * diff --git a/toolkit/mozapps/extensions/test/browser/head.js b/toolkit/mozapps/extensions/test/browser/head.js @@ -631,7 +631,7 @@ function addCertOverrides() { ]); } -/** *** Mock Provider *****/ +/***** Mock Provider *****/ function MockProvider( addonTypes, @@ -683,7 +683,7 @@ MockProvider.prototype = { } }, - /** *** Utility functions *****/ + /***** Utility functions *****/ /** * Register this provider with the AddonManager @@ -882,7 +882,7 @@ MockProvider.prototype = { return newInstalls; }, - /** *** AddonProvider implementation *****/ + /***** AddonProvider implementation *****/ /** * Called to initialize the provider. @@ -1036,7 +1036,7 @@ MockProvider.prototype = { }, }; -/** *** Mock Addon object for the Mock Provider *****/ +/***** Mock Addon object for the Mock Provider *****/ function MockAddon(aId, aName, aType, aOperationsRequiringRestart) { // Only set required attributes. @@ -1294,7 +1294,7 @@ MockAddon.prototype = { }, }; -/** *** Mock AddonInstall object for the Mock Provider *****/ +/***** Mock AddonInstall object for the Mock Provider *****/ function MockInstall(aName, aType, aAddonToInstall) { this.name = aName || "";