1979318 - Change reporting of sap.counts source field to match SERP events and other telemetry

Status: RESOLVED FIXED

(Firefox :: Search, task, P2)

Description

[Mark Banner (:standard8)]

When reporting the source for a search, most of our telemetry is using source names such as urlbar_persisted and urlbar_handoff, however the sap.counts event telemetry is using urlbar-persisted and urlbar-handoff. This causes us to need additional mappings of one to another.

Currently, the sap.deprecatedCounts also uses the underscore version. However, that is supporting legacy telemetry, and would be best not changed.

Given that sap.counts was only added in Firefox 138, and isn't being used for sources just yet, I would like to change the source reporting to be consistent with the rest of the telemetry. Once we drop sap.deprecatedCounts, we'll also be able to remove the mapping and simplify the code.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1979318 - Change reporting of sap.counts source field to match SERP events and other telemetry. r?#search-reviewers!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/a28a3b5a4f5d
https://hg.mozilla.org/integration/autoland/rev/661e8ff318ab
Change reporting of sap.counts source field to match SERP events and other telemetry. r=search-reviewers,scunnane

Comment 3

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/661e8ff318ab