Commit Graph

206 Commits (fenix/118.2.0)

Author SHA1 Message Date
William Lachance adb10fab4e [fenix] No issue - Stop generating `metrics.md` and direct to Glean Dictionary instead (https://github.com/mozilla-mobile/fenix/pull/19411)
Now that the Glean Dictionary (https://dictionary.telemetry.mozilla.org)
has hit production, our guidance is to move away from generating
metrics.md files. The Glean Dictionary is a better solution for this task
for a couple of reasons:

1. The Glean Dictionary contains a complete reference of all metrics
and pings sent by an application (not just those defined in the
application itself, as is the case with the markdown documentation)
2. The Glean Dictionary provides additional context and links beyond
what is provided by the documentation (for example, information on
how to access metric information via BigQuery or GLAM) as well as
annotations provided by data scientists in the course of analyzing data
(see, for example, the commentary section for:
https://dictionary.telemetry.mozilla.org/apps/fenix/metrics/metrics_search_count).

This should also reduce some of the churn/burden on people when
they submit PRs, as there is no longer any need to update metrics.md
as a part of them.
3 years ago
Roger Yang 2d709b67a7 [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/19147: Move startup metrics to right after Glean initialization (https://github.com/mozilla-mobile/fenix/pull/19252) 3 years ago
Mihai Adrian Carare 0e15a9668b [fenix] For https://github.com/mozilla-mobile/fenix/pull/19243 - Update metrics documentation. (https://github.com/mozilla-mobile/fenix/pull/19269) 3 years ago
mcarare 6774f41d26 [fenix] For https://github.com/mozilla-mobile/fenix/issues/18852: Add metrics for default browser settings experiment. 3 years ago
Michael Comella c7f94915cc [fenix] For https://github.com/mozilla-mobile/fenix/issues/18836: add metric perf.startup.startup_type. 3 years ago
Arturo Mejia 2119cfadae [fenix] Telemetry for https://github.com/mozilla-mobile/fenix/issues/18855, https://github.com/mozilla-mobile/fenix/issues/18851 and https://github.com/mozilla-mobile/fenix/issues/18857
For https://github.com/mozilla-mobile/fenix/issues/18857 [Telemetry] Send a Glean event when users change their default browser

For https://github.com/mozilla-mobile/fenix/issues/18855 [Telemetry] Send an event when users open the toolbar menu

For https://github.com/mozilla-mobile/fenix/issues/18851 [Telemetry] Send an event when users click on the "set as default browser" entry in the toolbar menu
3 years ago
Roger Yang 4f9a341746 [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/18948: Update tabstray telemetry (https://github.com/mozilla-mobile/fenix/pull/19004) 3 years ago
mcarare 8e8a872127 [fenix] For https://github.com/mozilla-mobile/fenix/issues/18854: Add metrics for home screen being displayed. 3 years ago
mcarare 9dede18ff5 [fenix] For https://github.com/mozilla-mobile/fenix/issues/18853: Add metrics for default browser new tab experiment. 3 years ago
mcarare 4ff47f6c1d [fenix] For https://github.com/mozilla-mobile/fenix/issues/18856: Add metrics for opening settings screen from home menu. 3 years ago
Michael Comella 073a4dc221 [fenix] For https://github.com/mozilla-mobile/fenix/issues/18836: note refactor in metrics docs. 3 years ago
Michael Comella c04e4f392d [fenix] For https://github.com/mozilla-mobile/fenix/issues/18836: replace StartupActivityStateProvider with StartupStateProvider.
The StartupActivityStateProvider uses an imperative implementation,
driven by callbacks, to set the state of the application. This is hard
to follow as you need to understand which callbacks will be called in
which order. For example, to make sense of an implementation like this,
COLD, WARM, AND HOT would likely need to be implemented in separate
ActivityLifecycleCallbacks.

I feel the StartupStateProvider is an improvement because it leverages
the StartupActivityLog to query a linear state for a more understandable
implementation. Furthermore, it seems accessible to write COLD, WARM,
and HOT in the same class because they can all be approached the same
way.
3 years ago
Sebastian Kaspari a23da507c5 [fenix] Introduce process lifecycle observer to collect metrics about tabs when app goes to foreground/background. 3 years ago
Mihai Adrian Carare 1e26be9dfb [fenix] For https://github.com/mozilla-mobile/fenix/issues/17361 - Update metrics documentation. (https://github.com/mozilla-mobile/fenix/pull/18842) 3 years ago
Michael Comella b26d782c2f [fenix] For https://github.com/mozilla-mobile/fenix/issues/18426: add data review links. 3 years ago
Michael Comella 9c3ad5608a [fenix] For https://github.com/mozilla-mobile/fenix/issues/18426: warn about flaw in `app_opened_all_startup` probe. 3 years ago
Michael Comella c5bfc690d7 [fenix] For https://github.com/mozilla-mobile/fenix/issues/18426: record cold start duration telemetry. 3 years ago
Michael Comella f5accce70f [fenix] For https://github.com/mozilla-mobile/fenix/issues/18426: add StartupActivityStateProvider.
This will be used to filter conditions for the start up telemetry we're
adding.
3 years ago
Michael Comella ee31fe29c2 [fenix] For https://github.com/mozilla-mobile/fenix/issues/18426: add ApplicationStartReasonProvider.
We need to know this state to add a correct COLD MAIN probe.
3 years ago
Roger Yang e1404fab67 [fenix] For https://github.com/mozilla-mobile/fenix/issues/17644: Record when user taps on a add-on's setting (https://github.com/mozilla-mobile/fenix/pull/18504) 3 years ago
Michael Comella 3b2bda7c81 [fenix] For https://github.com/mozilla-mobile/fenix/issues/18426: add measurement probes for long-running lifecycle methods.
In addition to the existing probes for onCreate, this should cover all
lifecycle methods that take a long time in our profiles.
3 years ago
Michael Comella 1f67254b10 [fenix] For https://github.com/mozilla-mobile/fenix/issues/18426: add telemetry for app.onCreate subsections.
Hopefully this will help us understand behavior of the
`application_on_create` probe, specifically that it seems to take longer
in telemetry than in does locally compared to `home_activity_on_create`
(comparing the medians to local runs)..
3 years ago
Mugurell 78b7d0e95c [fenix] For https://github.com/mozilla-mobile/fenix/issues/17869 - New AndroidKeystoreExperiment telemetry 3 years ago
mcarare 1c8387fdb9 [fenix] For https://github.com/mozilla-mobile/fenix/issues/17768: Renew telemetry expiring in March. 3 years ago
mcarare ebde904cfd [fenix] For https://github.com/mozilla-mobile/fenix/issues/18076: Renew telemetry expiring in April. 3 years ago
Roger Yang 1ca7dc61ab [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/18178: Rework History Telemetry (https://github.com/mozilla-mobile/fenix/pull/18261) 3 years ago
Roger Yang f0dcb64716 [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/18173: Add Telemetry When User Opens a Bookmark (https://github.com/mozilla-mobile/fenix/pull/18174) 3 years ago
Roger Yang b8342ac6f0 [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/18068: Use AwesomeBarFacts for AwesomeBar Telemetry (https://github.com/mozilla-mobile/fenix/pull/18090) 3 years ago
Roger Yang 70329154e5 [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/18163: Use SyncedTabsFacts for Synced Tab Telemetry (https://github.com/mozilla-mobile/fenix/pull/18172) 3 years ago
mcarare 23a1973b68 [fenix] Update documentation after PR 18257. 3 years ago
Roger Yang b04588c784 [fenix] Update metrics.md (https://github.com/mozilla-mobile/fenix/pull/18259) 3 years ago
Sebastian Kaspari 1368a9bd19 [fenix] Fix metric types of engine telemetry. 3 years ago
Michael Comella a0f46da9ad [fenix] For https://github.com/mozilla-mobile/fenix/issues/18157: convert clock_ticks_per_second to quantity type.
While we could easily move this into the metrics ping, it's better to
leave it in the other ping because it's less work and because (I think)
we'll be better able to match `framework_secondary` values to the clock
ticks if we combine them in the same ping.
3 years ago
Roger Yang 10e8862ed6 [fenix] Update metrics.md for perf startup entries (https://github.com/mozilla-mobile/fenix/pull/18162) 3 years ago
Michael Comella 188fe713ed [fenix] For https://github.com/mozilla-mobile/fenix/issues/17972: split frameworkStart into two metrics.
We do this in order to make it easier to analyze in GLAM: see the metric
descriptions for more details.

Additionally, we change the time unit to milliseconds to make it easier
to analyze in GLAM.
3 years ago
Michael Comella 3ae7379a5f [fenix] For https://github.com/mozilla-mobile/fenix/issues/17969: add duration probes for App.onCreate and HomeActivity.onCreate. 3 years ago
Roger Yang 998a10cdc9 [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/17089: Add metric to track both normal and private URI opened (https://github.com/mozilla-mobile/fenix/pull/17935) 3 years ago
Sebastian Kaspari 5fec5b8d61 [fenix] Issue https://github.com/mozilla-mobile/fenix/issues/12731: Collect telemetry about the content process of tabs getting killed. 3 years ago
Codrut Topliceanu 942195bfae [fenix] For https://github.com/mozilla-mobile/fenix/issues/17418 - Added telemetry for Google Default Top Site (https://github.com/mozilla-mobile/fenix/pull/17637)
* For https://github.com/mozilla-mobile/fenix/issues/17418 - Adds channel "ts" to TrackKey

This is used to track if the `InContentTelemetry` is a result of the user using the Google Top Site. It looks for `&channel=ts` within the uri.

* For https://github.com/mozilla-mobile/fenix/issues/17418 - Adds TopSite PerformedSearch back in

* For https://github.com/mozilla-mobile/fenix/issues/17418 - Check now looks for equality with GOOGLE_URL

* For https://github.com/mozilla-mobile/fenix/issues/17418 - Adds test for topSite changes
3 years ago
Gabriel Luong 1430e127b1 [fenix] For https://github.com/mozilla-mobile/fenix/issues/17418 - Add event ping telemetry for the Google Top Site click (https://github.com/mozilla-mobile/fenix/pull/17862)
Co-authored-by: codrut.topliceanu <codrut.topliceanu@softvision.ro>
Co-authored-by: Gabriel Luong <gabriel.luong@gmail.com>
3 years ago
Elise Richards b41d6160e5 [fenix] Increase date on expired metrics (https://github.com/mozilla-mobile/fenix/pull/17766) 3 years ago
Michael Comella 76864a8ece [fenix] For https://github.com/mozilla-mobile/fenix/issues/17553: renew storage.stats metrics. 3 years ago
Codrut Topliceanu 5e78e43440 [fenix] For https://github.com/mozilla-mobile/fenix/issues/16828 - Adds telemetry for open in app banner (https://github.com/mozilla-mobile/fenix/pull/17049)
* For https://github.com/mozilla-mobile/fenix/issues/16828 - Adds telemetry for open in app banner

Specifically: Banner displayed, Dismiss action, Go to Settings action.
3 years ago
Elise Richards fcf3c5f57a [fenix] For https://github.com/mozilla-mobile/fenix/issues/16485: Tabs tray grid view telemetry (https://github.com/mozilla-mobile/fenix/pull/17442)
* Tabs tray CFR telemetry interactions

* Create CFR events and implementation

* Tests for tab tray controller
3 years ago
Codrut Topliceanu 2f57dcfdac [fenix] For https://github.com/mozilla-mobile/fenix/issues/17481 - Remove unused long_press_tapped metric (https://github.com/mozilla-mobile/fenix/pull/17544) 3 years ago
Codrut Topliceanu b71d813d64 [fenix] For https://github.com/mozilla-mobile/fenix/issues/11580 - Tracks text selection context menu usage (https://github.com/mozilla-mobile/fenix/pull/16968)
* For https://github.com/mozilla-mobile/fenix/issues/11580 - Tracks text selection context menu usage

Tracks Copy, Search, Select All and Share items from the text selection context menu. Uses AC's DefaultSelectionActionDelegate to achieve this.

Co-authored-by: Gabriel Luong <gabriel.luong@gmail.com>
3 years ago
Mugurell 4ee2151e75 [fenix] For https://github.com/mozilla-mobile/fenix/issues/11578 - Cleanup download telemetry - metrics ping reference
These pings were implemented first as counters.
When changing to events "send_in_pings: - metrics" still remained causing
confusion.
This patch comes to remove this confusion and potential future issues.
4 years ago
Mugurell a50340fd25 [fenix] For https://github.com/mozilla-mobile/fenix/issues/15367 - Cleanup metrics ping reference (https://github.com/mozilla-mobile/fenix/pull/17071)
These pings were implemented first as counters.
When changing to events "send_in_pings: - metrics" still remained causing
confusion.
This patch comes to remove this confusion and potential future issues.
4 years ago
mcarare 78195c8368 [fenix] For https://github.com/mozilla-mobile/fenix/issues/16827: Send copy url metrics correctly. 4 years ago
Gabriel Luong 29e329d8f1 [fenix] For https://github.com/mozilla-mobile/fenix/issues/16941 - [Telemetry] Bookmark Counts (https://github.com/mozilla-mobile/fenix/pull/16942) 4 years ago