Closed Bug 1620656 Opened 5 months ago Closed 2 months ago

Remove GeckoView Telemetry Snapshotting during or after Firefox 79 (June-ish 2020)

Categories

(Toolkit :: Telemetry, task, P1)

79 Branch
task

Tracking

()

RESOLVED FIXED
mozilla79
Tracking Status
firefox79 --- fixed

People

(Reporter: chutten, Assigned: Dexter)

References

Details

(Whiteboard: [geckoview:m79][geckoview])

Attachments

(3 files)

bug 1620395 should have deprecated GeckoView Telemetry Snapshoting. Now that it's been three versions, we can (probably) remove it.

Double check with Focus that we can do this, then remove the API. Removing the code would be nice, too, but the biggest part is that we need to remember to disable it enough that it doesn't start the TelemetryGVIO thread: that's the whole reason we're doing this.

Whiteboard: [geckoview:m79]
See Also: → 1633036
See Also: → 1632740
Priority: P3 → --

This bug can be about disabling or removing, your choice.

Do remember to update the Histograms, Scalars, and Events documentation that a value of "geckoview" is now no longer allowed.

Assignee: nobody → alessio.placitelli
Severity: normal → S3
Status: NEW → ASSIGNED
Priority: -- → P1
Component: Telemetry → General
Product: Toolkit → GeckoView
Target Milestone: --- → mozilla79
Version: unspecified → 79 Branch
Component: General → Telemetry
Product: GeckoView → Toolkit
Whiteboard: [geckoview:m79] → [geckoview:m79][geckoview]
Depends on: 1644447

Talked with Jon and he opened an issue for Focus to track moving over to Glean.

The public API has been removed in bug 1644447. It should be safe to remove the backend implementation now.

This removes the support for the 'geckoview' telemetry in
the core and in the JSM files. It also cleans up testing
and the configurations.

Pushed by aplacitelli@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dd0ce5e4a1bf
Remove 'geckoview' telemetry support. r=chutten,geckoview-reviewers,aklotz
https://hg.mozilla.org/integration/autoland/rev/f0b502580ef0
Disable 'geckoview' in the registry files. r=chutten
https://hg.mozilla.org/integration/autoland/rev/af6b36a497e8
Fix tests broken by removing 'geckoview' telemetry. r=janerik
Flags: needinfo?(alessio.placitelli)
Pushed by aplacitelli@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5409134fc766
Remove 'geckoview' telemetry support. r=chutten,geckoview-reviewers,aklotz
https://hg.mozilla.org/integration/autoland/rev/d009c756c86b
Disable 'geckoview' in the registry files. r=chutten
https://hg.mozilla.org/integration/autoland/rev/dfab5bc68cde
Fix tests broken by removing 'geckoview' telemetry. r=janerik
You need to log in before you can comment on or make changes to this bug.