mozilla-mobile / fenix

⚠️ Fenix (Firefox for Android) moved to a new repository. It is now developed and maintained as part of: https://github.com/mozilla-mobile/firefox-android
https://github.com/mozilla-mobile/firefox-android
Mozilla Public License 2.0
6.47k stars 1.27k forks source link

[Telemetry] Add page load ms in baseline ping #1302

Closed bbinto closed 4 years ago

bbinto commented 5 years ago

Similar to Focus: https://github.com/mozilla-mobile/focus-android/blob/master/docs/Telemetry.md#page-load-time-histogram 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?

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

What probes (suggested, if applicable)

Dependencies (Added by PM and Eng)

Acceptance Criteria (Added by PM)

┆Issue is synchronized with this Jira Task

bbinto commented 5 years ago

@georgf - does GleanSDK supports histogram/aggregate numbers for this metric?

bbinto commented 5 years ago

Also @cpeterso maybe we should bring this up with the perf team if/how they want this metric and/or if this should be tracked somewhere else.

cpeterso commented 5 years ago

@vickychin says that she is talking to the telemetry team (?) about adding telemetry probes for Fenix page loads. I don't know if that is this GitHub issue or a different conversation.

bdekoz commented 5 years ago

I'm ok with the perf team tracking this here, and yes, this is the same issue we'd discussed in another perf meeting today.

bbinto commented 5 years ago

@bdekoz, @cpeterso and @vickychin

Ok, I'll close this here then since that's a request/requirement for the perf team, if you need the Fenix team to access or track this metric via the Fenix client, please re-open another ticket.