1840315 - [Splash screen] add telemetry to the experiment

Status: RESOLVED FIXED

(Firefox for Android :: Onboarding, task, P1)

Description

[Mike a [:mavduevskiy]]

follow up to 1833959. send telemetry when closing the splash screen

We should gather some telemetry:

first run splash screen background tasks completed: boolean true or false (should >always be sent, and set to true if the background tasks described above all complete >before the splash screen ends)

Comment 1

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1840315 – added telemetry to the splash screen data fetching (#2616)

Comment 2

[BMO Github Automation]

Authored by mike a
https://github.com/mozilla-mobile/firefox-android/commit/e5d17713f9ea29f7d7caafba150cb92ed8de1243
[main] Bug 1840315 – added telemetry to the splash screen data fetching

Comment 3

[Mira Lobontiu (Android QA)]

Verified as fixed on the latest Fenix Nightly 117.0a1 build from 7/6 with Google Pixel 6 (Android 13).
The splash-screen generated the following event:

{
          "timestamp": 5126,
          "category": "splash_screen",
          "name": "first_launch_extended",
          "extra": {
            "data_fetched": "true"
          }
        } 

Glean dashboard

Comment 4

[Roger Yang [:royang]]

Comment on attachment 9340904 [details] [review]
[mozilla-mobile/firefox-android] Bug 1840315 – added telemetry to the splash screen data fetching (#2616)

Beta/Release Uplift Approval Request

• User impact if declined: We will not collect telemetry for when splash screen is extended.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Telemetry change only
• String changes made/needed: N/A
• Is Android affected?: Yes

Comment 5

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1840315 – added telemetry to the splash screen data fetching (backport #2616) (#2930)

Comment 6

[Dianna Smith [:diannaS]]

Comment on attachment 9344892 [details] [review]
[mozilla-mobile/firefox-android] Bug 1840315 – added telemetry to the splash screen data fetching (backport #2616) (#2930)

Adding approval flag to the backport patch

Approved for Mobile 116.0b8

Comment 7

[BMO Github Automation]

Authored by https://github.com/mergify[bot]
https://github.com/mozilla-mobile/firefox-android/commit/5804077780873bd80a03199195ac62fbf5a646d9
[releases_v116] Bug 1840315 – added telemetry to the splash screen data fetching (#2930)

Comment 8

[Lorand Janos]

We have tried verifying the telemetry on the latest Beta 116.0b8 build, but without success.
Telemetry opens Fenix without the splash art logo, it looks like it's trying to open the process as soon as possible. While navigating through logcat, we've encountered the following message: "Splash Screen org.mozilla.firefox_beta (client)' was disposed without first being removed with the input manager!".
@Roger, if this still needs to be tested, could you help us find out where did we make a mistake or is there another way to verify it? Thank you!