Bug 1772100 - Part 18: Use plain object for lazy getter in browser/components/syncedtabs/. r=markh

Differential Revision: https://phabricator.services.mozilla.com/D148028
This commit is contained in:
Tooru Fujisawa
2022-06-06 22:46:25 +00:00
parent 7c2f1c68a6
commit 612a6d95b1
3 changed files with 11 additions and 5 deletions

View File

@@ -29,7 +29,9 @@ let { getChromeWindow } = ChromeUtils.import(
);
const { UIState } = ChromeUtils.import("resource://services-sync/UIState.jsm");
XPCOMUtils.defineLazyGetter(this, "FxAccountsCommon", function() {
const lazy = {};
XPCOMUtils.defineLazyGetter(lazy, "FxAccountsCommon", function() {
return ChromeUtils.import("resource://gre/modules/FxAccountsCommon.js");
});

View File

@@ -13,7 +13,9 @@ let log = ChromeUtils.import(
"resource://gre/modules/Log.jsm"
).Log.repository.getLogger("Sync.RemoteTabs");
XPCOMUtils.defineLazyModuleGetters(this, {
const lazy = {};
XPCOMUtils.defineLazyModuleGetters(lazy, {
OpenInTabsUtils: "resource:///modules/OpenInTabsUtils.jsm",
});
@@ -123,7 +125,7 @@ TabListComponent.prototype = {
},
onOpenTabs(urls, where) {
if (!OpenInTabsUtils.confirmOpenInTabs(urls.length, this._window)) {
if (!lazy.OpenInTabsUtils.confirmOpenInTabs(urls.length, this._window)) {
return;
}
if (where == "window") {

View File

@@ -6,8 +6,10 @@
const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
const lazy = {};
ChromeUtils.defineModuleGetter(
this,
lazy,
"PrivateBrowsingUtils",
"resource://gre/modules/PrivateBrowsingUtils.jsm"
);
@@ -555,7 +557,7 @@ TabListView.prototype = {
let show = false;
if (showTabOptions) {
if (el.getAttribute("id") == "syncedTabsOpenSelectedInPrivateWindow") {
show = PrivateBrowsingUtils.enabled;
show = lazy.PrivateBrowsingUtils.enabled;
} else if (
el.getAttribute("id") != "syncedTabsOpenAllInTabs" &&
el.getAttribute("id") != "syncedTabsManageDevices"