Bug 1700836 - Remove expired firstStartup telemetry. r=mythmon

Differential Revision: https://phabricator.services.mozilla.com/D114263

Author: agashlin

toolkit/components/telemetry/Scalars.yaml

@@ -6049,38 +6049,6 @@ blocklist:
     record_in_processes:
       - main
 
-firstStartup:
-  statusCode:
-    bug_numbers:
-      - 1582608
-    description: >-
-      Status of the FirstRun service, which runs post-install/early-startup in Firefox.
-    expires: "90"
-    kind: uint
-    notification_emails:
-      - rhelmer@mozilla.com
-    release_channel_collection: opt-out
-    products:
-      - 'firefox'
-      - 'thunderbird'
-    record_in_processes:
-      - main
-  elapsed:
-    bug_numbers:
-      - 1582608
-    description: >-
-      Number of milliseconds the FirstRun service took to run.
-    expires: "90"
-    kind: uint
-    notification_emails:
-      - rhelmer@mozilla.com
-    release_channel_collection: opt-out
-    products:
-      - 'firefox'
-      - 'thunderbird'
-    record_in_processes:
-      - main
-
 # The following section is for the "deletion-request" ping (bug 1585410). All the
 # extra identifiers other than `client_id` could be added here.
 deletion.request:

toolkit/modules/FirstStartup.jsm

@@ -18,7 +18,6 @@ XPCOMUtils.defineLazyModuleGetters(this, {
 });
 
 const PREF_TIMEOUT = "first-startup.timeout";
-const PROBE_NAME = "firstStartup";
 
 /**
  * Service for blocking application startup, to be used on the first install. The intended
@@ -79,9 +78,6 @@ var FirstStartup = {
     } else {
       this._state = this.UNSUPPORTED;
     }
-
-    Services.telemetry.scalarSet(`${PROBE_NAME}.statusCode`, this._state);
-    Services.telemetry.scalarSet(`${PROBE_NAME}.elapsed`, this.elapsed);
   },
 
   get state() {

toolkit/modules/docs/FirstStartup.rst

@@ -63,19 +63,10 @@ List of phases
 
   The time-out has been reached before startup tasks are complete.
 
-  This status code is reported to Telemetry via the ``firstStartup.statusCode``
-  scalar.
-
 ``FirstStartup.SUCCESS``
 
   All startup tasks have completed successfully, and application startup may resume.
 
-  This status code is reported to Telemetry via the ``firstStartup.statusCode``
-  scalar.
-
 ``FirstStartup.UNSUPPORTED``
 
   No startup tasks are supported, and `FirstStartup` exited.
-
-  This status code is reported to Telemetry via the ``firstStartup.statusCode``
-  scalar.

toolkit/modules/tests/xpcshell/test_firstStartup.js

@@ -11,7 +11,6 @@ const { updateAppInfo } = ChromeUtils.import(
 );
 
 const PREF_TIMEOUT = "first-startup.timeout";
-const PROBE_NAME = "firstStartup.statusCode";
 
 add_task(async function test_success() {
   updateAppInfo();
@@ -21,16 +20,6 @@ add_task(async function test_success() {
   } else {
     equal(FirstStartup.state, FirstStartup.UNSUPPORTED);
   }
-
-  const scalars = Services.telemetry.getSnapshotForScalars("main", false)
-    .parent;
-  ok(PROBE_NAME in scalars);
-
-  if (AppConstants.MOZ_NORMANDY) {
-    equal(scalars[PROBE_NAME], FirstStartup.SUCCESS);
-  } else {
-    equal(scalars[PROBE_NAME], FirstStartup.UNSUPPORTED);
-  }
 });
 
 add_task(async function test_timeout() {
@@ -43,13 +32,4 @@ add_task(async function test_timeout() {
   } else {
     equal(FirstStartup.state, FirstStartup.UNSUPPORTED);
   }
-
-  const scalars = Services.telemetry.getSnapshotForScalars("main", false)
-    .parent;
-  ok(PROBE_NAME in scalars);
-  if (AppConstants.MOZ_NORMANDY) {
-    equal(scalars[PROBE_NAME], FirstStartup.TIMED_OUT);
-  } else {
-    equal(scalars[PROBE_NAME], FirstStartup.UNSUPPORTED);
-  }
 });

toolkit/modules/tests/xpcshell/xpcshell.ini

@@ -63,6 +63,6 @@ reason = LOCALE is not defined without MOZ_UPDATER
 [test_EventEmitter.js]
 [test_ProfileAge.js]
 [test_firstStartup.js]
-skip-if = true || toolkit == 'android' # Bug 1700836 - telemetry probe expired
+skip-if = toolkit == 'android'
 [test_AllowedAppSources.js]
 skip-if = os != 'win' # Test of a Windows-specific feature