commit 3c2d0ad5612be9316eb1d94d951bb6316357be76 parent 6a49b6b15f8a8366e47385ab22cf7495a906f423 Author: Gijs Kruitbosch <gijskruitbosch@gmail.com> Date: Tue, 4 Nov 2025 17:05:56 +0000 Bug 1997684 - use moz-src for browser/components/topsites, r=mconley Differential Revision: https://phabricator.services.mozilla.com/D270934 Diffstat:
16 files changed, 86 insertions(+), 34 deletions(-)
diff --git a/browser/components/topsites/TopSites.sys.mjs b/browser/components/topsites/TopSites.sys.mjs @@ -5,9 +5,9 @@ import { getDomain, TippyTopProvider, -} from "resource:///modules/topsites/TippyTopProvider.sys.mjs"; +} from "moz-src:///browser/components/topsites/TippyTopProvider.sys.mjs"; import { Dedupe } from "resource:///modules/Dedupe.sys.mjs"; -import { TOP_SITES_MAX_SITES_PER_ROW } from "resource:///modules/topsites/constants.mjs"; +import { TOP_SITES_MAX_SITES_PER_ROW } from "moz-src:///browser/components/topsites/constants.mjs"; import { CUSTOM_SEARCH_SHORTCUTS, checkHasSearchEngine, diff --git a/browser/components/topsites/moz.build b/browser/components/topsites/moz.build @@ -4,7 +4,7 @@ # License, v. 2.0. If a copy of the MPL was not distributed with this # file, You can obtain one at http://mozilla.org/MPL/2.0/. -EXTRA_JS_MODULES.topsites += [ +MOZ_SRC_FILES += [ "constants.mjs", "TippyTopProvider.sys.mjs", "TopSites.sys.mjs", diff --git a/browser/components/topsites/test/unit/test_FaviconProvider.js b/browser/components/topsites/test/unit/test_FaviconProvider.js @@ -4,7 +4,7 @@ "use strict"; ChromeUtils.defineESModuleGetters(this, { - FaviconProvider: "resource:///modules/topsites/TopSites.sys.mjs", + FaviconProvider: "moz-src:///browser/components/topsites/TopSites.sys.mjs", HttpServer: "resource://testing-common/httpd.sys.mjs", NewTabUtils: "resource://gre/modules/NewTabUtils.sys.mjs", NetUtil: "resource://gre/modules/NetUtil.sys.mjs", diff --git a/browser/components/topsites/test/unit/test_TippyTopProvider.js b/browser/components/topsites/test/unit/test_TippyTopProvider.js @@ -5,7 +5,8 @@ ChromeUtils.defineESModuleGetters(this, { sinon: "resource://testing-common/Sinon.sys.mjs", - TippyTopProvider: "resource:///modules/topsites/TippyTopProvider.sys.mjs", + TippyTopProvider: + "moz-src:///browser/components/topsites/TippyTopProvider.sys.mjs", }); /** diff --git a/browser/components/topsites/test/unit/test_top_sites.js b/browser/components/topsites/test/unit/test_top_sites.js @@ -4,7 +4,9 @@ "use strict"; const { TopSites, insertPinned, DEFAULT_TOP_SITES } = - ChromeUtils.importESModule("resource:///modules/topsites/TopSites.sys.mjs"); + ChromeUtils.importESModule( + "moz-src:///browser/components/topsites/TopSites.sys.mjs" + ); ChromeUtils.defineESModuleGetters(this, { FilterAdult: "resource:///modules/FilterAdult.sys.mjs", @@ -15,8 +17,10 @@ ChromeUtils.defineESModuleGetters(this, { PlacesUtils: "resource://gre/modules/PlacesUtils.sys.mjs", SearchService: "resource://gre/modules/SearchService.sys.mjs", TestUtils: "resource://testing-common/TestUtils.sys.mjs", - TOP_SITES_DEFAULT_ROWS: "resource:///modules/topsites/constants.mjs", - TOP_SITES_MAX_SITES_PER_ROW: "resource:///modules/topsites/constants.mjs", + TOP_SITES_DEFAULT_ROWS: + "moz-src:///browser/components/topsites/constants.mjs", + TOP_SITES_MAX_SITES_PER_ROW: + "moz-src:///browser/components/topsites/constants.mjs", }); const FAKE_FAVICON = "data987"; diff --git a/browser/components/urlbar/UrlbarProviderTopSites.sys.mjs b/browser/components/urlbar/UrlbarProviderTopSites.sys.mjs @@ -18,9 +18,11 @@ const lazy = {}; ChromeUtils.defineESModuleGetters(lazy, { AboutNewTab: "resource:///modules/AboutNewTab.sys.mjs", PlacesUtils: "resource://gre/modules/PlacesUtils.sys.mjs", - TopSites: "resource:///modules/topsites/TopSites.sys.mjs", - TOP_SITES_DEFAULT_ROWS: "resource:///modules/topsites/constants.mjs", - TOP_SITES_MAX_SITES_PER_ROW: "resource:///modules/topsites/constants.mjs", + TopSites: "moz-src:///browser/components/topsites/TopSites.sys.mjs", + TOP_SITES_DEFAULT_ROWS: + "moz-src:///browser/components/topsites/constants.mjs", + TOP_SITES_MAX_SITES_PER_ROW: + "moz-src:///browser/components/topsites/constants.mjs", UrlbarPrefs: "moz-src:///browser/components/urlbar/UrlbarPrefs.sys.mjs", UrlbarProviderOpenTabs: "moz-src:///browser/components/urlbar/UrlbarProviderOpenTabs.sys.mjs", diff --git a/browser/components/urlbar/tests/browser/browser_top_sites.js b/browser/components/urlbar/tests/browser/browser_top_sites.js @@ -6,7 +6,7 @@ ChromeUtils.defineESModuleGetters(this, { AboutNewTab: "resource:///modules/AboutNewTab.sys.mjs", NewTabUtils: "resource://gre/modules/NewTabUtils.sys.mjs", - TopSites: "resource:///modules/topsites/TopSites.sys.mjs", + TopSites: "moz-src:///browser/components/topsites/TopSites.sys.mjs", }); const EN_US_TOPSITES = diff --git a/browser/components/urlbar/tests/browser/browser_top_sites_private.js b/browser/components/urlbar/tests/browser/browser_top_sites_private.js @@ -6,7 +6,7 @@ ChromeUtils.defineESModuleGetters(this, { AboutNewTab: "resource:///modules/AboutNewTab.sys.mjs", NewTabUtils: "resource://gre/modules/NewTabUtils.sys.mjs", - TopSites: "resource:///modules/topsites/TopSites.sys.mjs", + TopSites: "moz-src:///browser/components/topsites/TopSites.sys.mjs", }); const EN_US_TOPSITES = diff --git a/browser/components/urlbar/tests/browser/head-common.js b/browser/components/urlbar/tests/browser/head-common.js @@ -6,7 +6,7 @@ ChromeUtils.defineESModuleGetters(this, { PlacesUtils: "resource://gre/modules/PlacesUtils.sys.mjs", Preferences: "resource://gre/modules/Preferences.sys.mjs", sinon: "resource://testing-common/Sinon.sys.mjs", - TopSites: "resource:///modules/topsites/TopSites.sys.mjs", + TopSites: "moz-src:///browser/components/topsites/TopSites.sys.mjs", UrlbarProvider: "moz-src:///browser/components/urlbar/UrlbarUtils.sys.mjs", UrlbarProvidersManager: "moz-src:///browser/components/urlbar/UrlbarProvidersManager.sys.mjs", diff --git a/browser/extensions/newtab/common/Reducers.sys.mjs b/browser/extensions/newtab/common/Reducers.sys.mjs @@ -4,11 +4,19 @@ import { actionTypes as at } from "resource://newtab/common/Actions.mjs"; import { Dedupe } from "resource:///modules/Dedupe.sys.mjs"; +import { ImportHelper } from "resource://newtab/lib/ImportHelper.sys.mjs"; -export { - TOP_SITES_DEFAULT_ROWS, - TOP_SITES_MAX_SITES_PER_ROW, -} from "resource:///modules/topsites/constants.mjs"; +/** + * @backward-compat { version 146 } + * + * Switch back to aggregate export for these constants once 146 hits release. + */ +const { TOP_SITES_DEFAULT_ROWS, TOP_SITES_MAX_SITES_PER_ROW } = + ImportHelper.import( + "moz-src:///browser/components/topsites/constants.mjs", + "resource:///modules/topsites/" + ); +export { TOP_SITES_DEFAULT_ROWS, TOP_SITES_MAX_SITES_PER_ROW }; const dedupe = new Dedupe(site => site && site.url); diff --git a/browser/extensions/newtab/lib/FaviconFeed.sys.mjs b/browser/extensions/newtab/lib/FaviconFeed.sys.mjs @@ -3,7 +3,17 @@ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ import { actionTypes as at } from "resource://newtab/common/Actions.mjs"; -import { FaviconProvider } from "resource:///modules/topsites/TopSites.sys.mjs"; +import { ImportHelper } from "resource://newtab/lib/ImportHelper.sys.mjs"; + +/** + * @backward-compat { version 146 } + * + * Switch back to a normal import once 146 hits release. + */ +const { FaviconProvider } = ImportHelper.import( + "moz-src:///browser/components/topsites/TopSites.sys.mjs", + "resource:///modules/topsites/" +); export class FaviconFeed { constructor() { diff --git a/browser/extensions/newtab/lib/HighlightsFeed.sys.mjs b/browser/extensions/newtab/lib/HighlightsFeed.sys.mjs @@ -4,10 +4,18 @@ import { actionTypes as at } from "resource://newtab/common/Actions.mjs"; -import { - TOP_SITES_DEFAULT_ROWS, - TOP_SITES_MAX_SITES_PER_ROW, -} from "resource:///modules/topsites/constants.mjs"; +import { ImportHelper } from "resource://newtab/lib/ImportHelper.sys.mjs"; + +/** + * @backward-compat { version 146 } + * + * Switch back to a normal import once 146 hits release. + */ +const { TOP_SITES_DEFAULT_ROWS, TOP_SITES_MAX_SITES_PER_ROW } = + ImportHelper.import( + "moz-src:///browser/components/topsites/constants.mjs", + "resource:///modules/topsites/" + ); import { Dedupe } from "resource:///modules/Dedupe.sys.mjs"; const lazy = {}; diff --git a/browser/extensions/newtab/lib/TopSitesFeed.sys.mjs b/browser/extensions/newtab/lib/TopSitesFeed.sys.mjs @@ -6,9 +6,26 @@ import { actionCreators as ac, actionTypes as at, } from "resource://newtab/common/Actions.mjs"; -import { TippyTopProvider } from "resource:///modules/topsites/TippyTopProvider.sys.mjs"; -import { insertPinned } from "resource:///modules/topsites/TopSites.sys.mjs"; -import { TOP_SITES_MAX_SITES_PER_ROW } from "resource:///modules/topsites/constants.mjs"; + +import { ImportHelper } from "resource://newtab/lib/ImportHelper.sys.mjs"; + +/** + * @backward-compat { version 146 } + * + * Switch back to normal imports once 146 hits release. + */ +const { TippyTopProvider } = ImportHelper.import( + "moz-src:///browser/components/topsites/TippyTopProvider.sys.mjs", + "resource:///modules/topsites/" +); +const { insertPinned } = ImportHelper.import( + "moz-src:///browser/components/topsites/TopSites.sys.mjs", + "resource:///modules/topsites/" +); +const { TOP_SITES_MAX_SITES_PER_ROW } = ImportHelper.import( + "moz-src:///browser/components/topsites/constants.mjs", + "resource:///modules/topsites/" +); import { Dedupe } from "resource:///modules/Dedupe.sys.mjs"; import { diff --git a/browser/extensions/newtab/test/xpcshell/test_TopSitesFeed.js b/browser/extensions/newtab/test/xpcshell/test_TopSitesFeed.js @@ -16,8 +16,10 @@ ChromeUtils.defineESModuleGetters(this, { Screenshots: "resource://newtab/lib/Screenshots.sys.mjs", Sampling: "resource://gre/modules/components-utils/Sampling.sys.mjs", SearchService: "resource://gre/modules/SearchService.sys.mjs", - TOP_SITES_DEFAULT_ROWS: "resource:///modules/topsites/constants.mjs", - TOP_SITES_MAX_SITES_PER_ROW: "resource:///modules/topsites/constants.mjs", + TOP_SITES_DEFAULT_ROWS: + "moz-src:///browser/components/topsites/constants.mjs", + TOP_SITES_MAX_SITES_PER_ROW: + "moz-src:///browser/components/topsites/constants.mjs", TopSitesFeed: "resource://newtab/lib/TopSitesFeed.sys.mjs", }); diff --git a/tools/@types/generated/lib.gecko.modules.d.ts b/tools/@types/generated/lib.gecko.modules.d.ts @@ -509,9 +509,9 @@ export interface Modules { "resource:///modules/taskbartabs/TaskbarTabsRegistry.sys.mjs": typeof import("resource:///modules/taskbartabs/TaskbarTabsRegistry.sys.mjs"), "resource:///modules/taskbartabs/TaskbarTabsUtils.sys.mjs": typeof import("resource:///modules/taskbartabs/TaskbarTabsUtils.sys.mjs"), "resource:///modules/taskbartabs/TaskbarTabsWindowManager.sys.mjs": typeof import("resource:///modules/taskbartabs/TaskbarTabsWindowManager.sys.mjs"), - "resource:///modules/topsites/TippyTopProvider.sys.mjs": typeof import("resource:///modules/topsites/TippyTopProvider.sys.mjs"), - "resource:///modules/topsites/TopSites.sys.mjs": typeof import("resource:///modules/topsites/TopSites.sys.mjs"), - "resource:///modules/topsites/constants.mjs": typeof import("resource:///modules/topsites/constants.mjs"), + "moz-src:///browser/components/topsites/TippyTopProvider.sys.mjs": typeof import("moz-src:///browser/components/topsites/TippyTopProvider.sys.mjs"), + "moz-src:///browser/components/topsites/TopSites.sys.mjs": typeof import("moz-src:///browser/components/topsites/TopSites.sys.mjs"), + "moz-src:///browser/components/topsites/constants.mjs": typeof import("moz-src:///browser/components/topsites/constants.mjs"), "resource:///modules/webrtcUI.sys.mjs": typeof import("resource:///modules/webrtcUI.sys.mjs"), "resource://autofill/FormAutofill.sys.mjs": typeof import("resource://autofill/FormAutofill.sys.mjs"), "resource://autofill/FormAutofillChild.ios.sys.mjs": typeof import("resource://autofill/FormAutofillChild.ios.sys.mjs"), diff --git a/tools/@types/generated/tspaths.json b/tools/@types/generated/tspaths.json @@ -1421,13 +1421,13 @@ "resource:///modules/taskbartabs/TaskbarTabsWindowManager.sys.mjs": [ "browser/components/taskbartabs/TaskbarTabsWindowManager.sys.mjs" ], - "resource:///modules/topsites/TippyTopProvider.sys.mjs": [ + "moz-src:///browser/components/topsites/TippyTopProvider.sys.mjs": [ "browser/components/topsites/TippyTopProvider.sys.mjs" ], - "resource:///modules/topsites/TopSites.sys.mjs": [ + "moz-src:///browser/components/topsites/TopSites.sys.mjs": [ "browser/components/topsites/TopSites.sys.mjs" ], - "resource:///modules/topsites/constants.mjs": [ + "moz-src:///browser/components/topsites/constants.mjs": [ "browser/components/topsites/constants.mjs" ], "resource:///modules/webrtcUI.sys.mjs": [