[Telemetry] Add uri_count to ping

[bbinto]

Similar to Focus: https://github.com/mozilla-mobile/focus-android/blob/master/docs/Telemetry.md#uri-count

What questions will you answer with this data?

• As a product manager, I want to know how many websites URIs are being visited per session

Why does Mozilla need to answer these questions? Are there benefits for users? Do we need this information to address product or business requirements?

• The amount of URI count is a good indicator for us to know how useful and sticky the app is

What probes (suggested, if applicable)

• URI count, see row 30 in https://docs.google.com/spreadsheets/d/1P2_aGEE9OtjM3x71d0GBhnHDhzxEL02xEHuKn1no23E/edit#gid=0
• similar to Focus

Dependencies (Added by PM and Eng)

• Needs a ping which resets per session

Acceptance Criteria (Added by PM)

• Baseline pings can be queried via re:dash
• We are sending telemetry events for the actions listed in the requirements
• We have documented the telemetry
• We have asked a data steward to review the telemetry
• NOT an AC: Data science to create dashboard or further graphs (this will be a separate issue, this issue is only about hooking up the events described and that they can be queried in re-dash)

┆Issue is synchronized with this Jira Task

[boek]

This can be done pretty easily with the counter metric type

[bsurd]

The total_uri_count is not displayed in the events when sending telemetry. Looked in both baseline and events JSON files and the data wasn't available anywhere.

Started the app with glean via cmd and I've tried a scenario where I visited multiple different pages in different tabs, one where I visited multiple pages in the same tab and one where I navigated from one link to another in the same tab.

Please let me know if I've missed anything.

[vesta0]

@colintheshots I re-added the QA needed label based on your comment.

[AndiAJ]

Hi all, the uri_count total_uri_count is now sent with the baseline ping, it works.

Please find attached and review the following:

•1 search performed: Logcat: UriCount3.txt
Glean Dashboard:https://debug-ping-preview.firebaseapp.com/pings/2311bde3-47fc-4272-9a8b-b1cce4e7eaf5/aj-uritest-04

•4 searches performed: Logcat: UriCount1.txt
Glean Dashboard: Glean Dashboard:https://debug-ping-preview.firebaseapp.com/pings/2311bde3-47fc-4272-9a8b-b1cce4e7eaf5/aj-uritest-00

•8 searches performed: Logcat: UriCount2.txt
Glean Dashboard: (second baseline) https://debug-ping-preview.firebaseapp.com/pings/2311bde3-47fc-4272-9a8b-b1cce4e7eaf5/aj-uritest-03

Before closing this ticket, in order to have a better understanding regarding thesearch_count VS uri_count I would like to ask, why the uri_count is always higher than the search_count ? what is the difference between the two?
Please review the above attached logs and confirm if from your point of view the uri_count works properly.

Until then, based on this comment I'll remove the QA needed label.

[sblatz]

@colintheshots or @boek are you able to explain this behavior?

[boek]

Looks like there is a bug that is causing UriOpened to get called an extra time in a session. I filed a follow up bug #3676 and we can track/fix it there.

[boek]

@WhoIsAndi is that good enough to close this issue?

[AndiAJ]

@boek sure thing ! I'll go ahead and close this issue.