Bug 1435231 - Initialize Normandy events before other subcomponents r?Gijs draft
authorMike Cooper <mcooper@mozilla.com>
Tue, 06 Feb 2018 14:49:59 -0800
changeset 751808 4198f34e5d0b10d1a071f17898e22a1a022f1ca1
parent 751641 c6852f03a2f32029c04735687a4bc13c3d467ab5
push id98058
push userbmo:mcooper@mozilla.com
push dateTue, 06 Feb 2018 22:50:13 +0000
reviewersGijs
bugs1435231
milestone60.0a1
Bug 1435231 - Initialize Normandy events before other subcomponents r?Gijs MozReview-Commit-ID: KcMQF5V6Hxl
browser/extensions/shield-recipe-client/lib/ShieldRecipeClient.jsm
--- a/browser/extensions/shield-recipe-client/lib/ShieldRecipeClient.jsm
+++ b/browser/extensions/shield-recipe-client/lib/ShieldRecipeClient.jsm
@@ -43,16 +43,22 @@ this.ShieldRecipeClient = {
     LogManager.configure(Services.prefs.getIntPref(PREF_LOGGING_LEVEL));
     Services.prefs.addObserver(PREF_LOGGING_LEVEL, LogManager.configure);
     CleanupManager.addCleanupHandler(
       () => Services.prefs.removeObserver(PREF_LOGGING_LEVEL, LogManager.configure),
     );
     log = LogManager.getLogger("bootstrap");
 
     try {
+      TelemetryEvents.init();
+    } catch (err) {
+      log.error("Failed to initialize telemetry events:", err);
+    }
+
+    try {
       await AboutPages.init();
     } catch (err) {
       log.error("Failed to initialize about pages:", err);
     }
 
     try {
       await AddonStudies.init();
     } catch (err) {
@@ -66,22 +72,16 @@ this.ShieldRecipeClient = {
     }
 
     try {
       ShieldPreferences.init();
     } catch (err) {
       log.error("Failed to initialize preferences UI:", err);
     }
 
-    try {
-      TelemetryEvents.init();
-    } catch (err) {
-      log.error("Failed to initialize telemetry events:", err);
-    }
-
     await RecipeRunner.init();
     Services.obs.notifyObservers(null, SHIELD_INIT_NOTIFICATION);
   },
 
   async shutdown(reason) {
     await CleanupManager.cleanup();
   },
 };