1888012 - [toolbar redesign] verify that no telemetry is lost after redesign

Status: NEW

(Fenix :: Toolbar, task, P3)

Description

[Mike a [:mavduevskiy]]

We want to make sure that no telemetry is missing after the redesign. The new design didn't introduce a lot of new events, but mostly reusing already existing ones. For example, navigation bar elements (back/forth/home/tabs/menu) exist in the current design as well, and we should make sure that the same events are triggered when user interacts with the nav bar as it was prior to the redesign.

Comment 1

[Roger Yang [:royang]]

Pleas also make sure that home, back, forward, search, voice and QR telemetry have source information so we know where users are coming from.

Comment 2

[Roger Yang [:royang]]

In case of toolbar redesign, we will now be sending two telemetry related to QR/Voice. Once above where you added

if (activateQrReader) {
            Events.searchBarTapped.record(Events.SearchBarTappedExtra("HOME_QR"))
        } else if (activateVoiceSearch) {
            Events.searchBarTapped.record(Events.SearchBarTappedExtra("HOME_VOICE"))
}

And once in these functions of launchQr() and launchVoiceSearch() where we have

Events.browserToolbarQrScanTapped.record(NoExtras())

VoiceSearch.tapped.record(NoExtras())

We should update these.

Comment 3

[Roger Yang [:royang]]

One thing missing is we need to add telemetry for the new two buttons, share and reload.

Comment 4

[Chris Peterson [:cpeterson]]

Setting toolbar redesign's Release blockers to priority P3 until we finish fixing our Beta experiment blockers.