Skip to content

Commit

Permalink
issue #6133: Rename tab count ping
Browse files Browse the repository at this point in the history
  • Loading branch information
dnarcese committed Mar 19, 2020
1 parent 2b3ff6e commit 47a8e07
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions Client/Telemetry/UnifiedTelemetry.swift
Original file line number Diff line number Diff line change
Expand Up @@ -116,9 +116,9 @@ class UnifiedTelemetry {
outputDict["settings"] = settings

let delegate = UIApplication.shared.delegate as? AppDelegate
outputDict["tabs"] = delegate?.tabManager.count ?? 0

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?]] {
Expand Down
2 changes: 1 addition & 1 deletion Docs/telemetry.md
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ The defaultMailClient field contains the URL scheme of the mail client that the

The defaultNewTabExperience field contains the name of the view that the user wants to see on new tabs. For example History, Homepage or Blank. It is used to measure usage of this feature, to see how effective feature promotion campaigns are and to establish a baseline number for when we introduce the new Activity Stream features. Duration: There is no intent to remove this field.

- tabs
- openTabCount

The number of tabs the user had open during a unique session.

Expand Down

0 comments on commit 47a8e07

Please sign in to comment.