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

1 files changed, 0 insertions, 12 deletions

diff --git a/toolkit/mozapps/extensions/internal/XPIProvider.jsm b/toolkit/mozapps/extensions/internal/XPIProvider.jsm
index 3bcb4275eb..dbb1a18dd1 100644
--- a/toolkit/mozapps/extensions/internal/XPIProvider.jsm
+++ b/toolkit/mozapps/extensions/internal/XPIProvider.jsm
@@ -205,14 +205,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
@@ -1832,8 +1824,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: {},
@@ -2007,7 +1997,6 @@ this.XPIProvider = {
 
     try {
       logger.debug("startup");
-      this.runPhase = XPI_STARTING;
       this.installs = [];
       this.installLocations = [];
       this.installLocationsByName = {};
@@ -2178,7 +2167,6 @@ this.XPIProvider = {
       }, "quit-application-granted", false);
 
       this.extensionsActive = true;
-      this.runPhase = XPI_BEFORE_UI_STARTUP;
     }
     catch (e) {
       logger.error("startup failed", e);