Fix #6392: Move openTabCount telemetry to core ping

mozilla-mobile · Apr 9, 2020 · ca632b8 · ca632b8
1 parent 0170f5f
commit ca632b8
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/Client/Telemetry/UnifiedTelemetry.swift b/Client/Telemetry/UnifiedTelemetry.swift
@@ -91,6 +91,10 @@ class UnifiedTelemetry {
 
             outputDict["settings"] = settings
 
+            let delegate = UIApplication.shared.delegate as? AppDelegate
+
+            outputDict["openTabCount"] = delegate?.tabManager.count ?? 0
+
             var userInterfaceStyle = "unknown" // unknown implies that device is on pre-iOS 13
             if #available(iOS 13.0, *) {
                 userInterfaceStyle = UITraitCollection.current.userInterfaceStyle == .dark ? "dark" : "light"
@@ -115,10 +119,6 @@ class UnifiedTelemetry {
 
             outputDict["settings"] = settings
 
-            let delegate = UIApplication.shared.delegate as? AppDelegate
-
-            outputDict["openTabCount"] = delegate?.tabManager.count ?? 0
-
             // App Extension telemetry requires reading events stored in prefs, then clearing them from prefs.
             if let extensionEvents = profile.prefs.arrayForKey(PrefsKeys.AppExtensionTelemetryEventArray) as? [[String: String]],
                 var pingEvents = outputDict["events"] as? [[Any?]] {