Fix mozilla-mobile#5730 - Private mode telemetry

garvankeeley · Nov 21, 2019 · c236537 · c236537
1 parent 46b6676
commit c236537
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 0 deletions.
diff --git a/Client/Frontend/Browser/Tab.swift b/Client/Frontend/Browser/Tab.swift
@@ -200,6 +200,8 @@ class Tab: NSObject {
         self.isPrivate = isPrivate
 
         debugTabCount += 1
+
+        UnifiedTelemetry.recordEvent(category: .action, method: .add, object: .tab, value: isPrivate ? .privateTab : .normalTab)
     }
 
     class func toRemoteTab(_ tab: Tab) -> RemoteTab? {

diff --git a/Client/Frontend/Browser/TabDisplayManager.swift b/Client/Frontend/Browser/TabDisplayManager.swift
@@ -123,6 +123,8 @@ class TabDisplayManager: NSObject {
         isPrivate = isOn
         UserDefaults.standard.set(isPrivate, forKey: "wasLastSessionPrivate")
 
+        UnifiedTelemetry.recordEvent(category: .action, method: .tap, object: .privateBrowsingButton)
+
         searchedTabs = nil
         refreshStore()
 

diff --git a/Client/Telemetry/UnifiedTelemetry.swift b/Client/Telemetry/UnifiedTelemetry.swift
@@ -194,6 +194,7 @@ extension UnifiedTelemetry {
         case dismissedOnboarding = "dismissed-onboarding"
         case dismissedOnboardingEmailLogin = "dismissed-onboarding-email-login"
         case dismissedOnboardingSignUp = "dismissed-onboarding-sign-up"
+        case privateBrowsingButton = "private-browsing-button"
     }
 
     public enum EventValue: String {
@@ -221,6 +222,8 @@ extension UnifiedTelemetry {
         case themeModeAutomatically = "theme-automatically"
         case themeLight = "theme-light"
         case themeDark = "theme-dark"
+        case privateTab = "private-tab"
+        case normalTab = "normal-tab"
     }
 
     public static func recordEvent(category: EventCategory, method: EventMethod, object: EventObject, value: EventValue, extras: [String: Any?]? = nil) {