From 06d8563a5809544b91528b2da15b5794b2e8a0b9 Mon Sep 17 00:00:00 2001 From: FranklinDM Date: Sun, 10 Apr 2022 00:22:25 +0800 Subject: [PATCH] Issue #21 - Remove run phase property and constants from XPIProvider.jsm These were introduced in Bug 853388 for the sole purpose of tracking the current startup phase for telemetry --- toolkit/mozapps/extensions/internal/XPIProvider.jsm | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/toolkit/mozapps/extensions/internal/XPIProvider.jsm b/toolkit/mozapps/extensions/internal/XPIProvider.jsm index 6e2ebb6269..690351be1d 100644 --- a/toolkit/mozapps/extensions/internal/XPIProvider.jsm +++ b/toolkit/mozapps/extensions/internal/XPIProvider.jsm @@ -206,14 +206,6 @@ const RESTARTLESS_TYPES = new Set([ "locale", ]); -// Keep track of where we are in startup. -// event happened during XPIDatabase.startup() -const XPI_STARTING = "XPIStarting"; -// event happened after startup() but before the final-ui-startup event -const XPI_BEFORE_UI_STARTUP = "BeforeFinalUIStartup"; -// event happened after final-ui-startup -const XPI_AFTER_UI_STARTUP = "AfterFinalUIStartup"; - const COMPATIBLE_BY_DEFAULT_TYPES = { extension: true, dictionary: true @@ -1833,8 +1825,6 @@ this.XPIProvider = { allAppGlobal: true, // A string listing the enabled add-ons for annotating crash reports enabledAddons: null, - // Keep track of startup phases. - runPhase: XPI_STARTING, // Keep track of the newest file in each add-on, in case we want to // report it. _mostRecentlyModifiedFile: {}, @@ -2008,7 +1998,6 @@ this.XPIProvider = { try { logger.debug("startup"); - this.runPhase = XPI_STARTING; this.installs = []; this.installLocations = []; this.installLocationsByName = {}; @@ -2179,7 +2168,6 @@ this.XPIProvider = { }, "quit-application-granted", false); this.extensionsActive = true; - this.runPhase = XPI_BEFORE_UI_STARTUP; } catch (e) { logger.error("startup failed", e);