1666979 - Remove or update probes expiring in Firefox 84: PDF_VIEWER_*

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, task)

Description

[telemetry-probes]

The following Firefox probes will expire in the next major Firefox nightly release: version 84 [1].

PDF_VIEWER_DOCUMENT_VERSION_2
PDF_VIEWER_DOCUMENT_GENERATOR_2
PDF_VIEWER_DOCUMENT_SIZE_KB
PDF_VIEWER_FONT_TYPES_2
PDF_VIEWER_EMBED_2
PDF_VIEWER_FORM_2
PDF_VIEWER_STREAM_TYPES_2
PDF_VIEWER_FALLBACK_REASON
PDF_VIEWER_FALLBACK_ERROR
pdf.viewer.fallback_shown
pdf.viewer.print

What to do about this:

1. If one, some, or all of the metrics are no longer needed, please remove them from their definitions files (Histograms.json, Scalars.yaml, Events.yaml).
2. If one, some, or all of the metrics are still required, please submit a Data Collection Review [2] and patch to extend their expiry. There is a shorter form for data collection renewal [3].

If you have any problems, please ask for help on the #fx-metrics Slack channel or the #telemetry Matrix room at https://chat.mozilla.org/#/room/#telemetry:mozilla.org. We'll give you a hand.

Your Friendly, Neighborhood Telemetry Team

[1] https://wiki.mozilla.org/Release_Management/Calendar
[2] https://wiki.mozilla.org/Firefox/Data_Collection
[3] https://github.com/mozilla/data-review/blob/master/renewal_request.md

This is an automated message sent from probe-scraper. See https://github.com/mozilla/probe-scraper for details.

Comment 1

[Brendan Dahl [:bdahl]]

Attachment: Bug 1666979 - Extend pdf.js telemetry probes. r=chutten

Work on the PDF viewer is continuing so we'd like to keep tracking this
info as we land more changes.

Comment 2

[Brendan Dahl [:bdahl]]

Attachment: renewal_request.txt

Comment 3

[Chris H-C :chutten]

Comment on attachment 9193551 [details]
renewal_request.txt

Is the provided Data Collection Review complete, correct, and data-review+ by a Data Steward?

Yes.

Is the data collection covered by the existing Firefox Privacy Notice?

Yes.

---

Result: datareview+

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fd9ecadf0baf
Extend pdf.js telemetry probes. r=chutten

Comment 5

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/fd9ecadf0baf

Comment 6

[Brendan Dahl [:bdahl]]

Comment on attachment 9193548 [details]
Bug 1666979 - Extend pdf.js telemetry probes. r=chutten

Beta/Release Uplift Approval Request

• User impact if declined: Miss some telemetry data.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• 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): This telemetry was previously enabled.
• String changes made/needed:

Comment 7

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9193548 [details]
Bug 1666979 - Extend pdf.js telemetry probes. r=chutten

Approved for 85.0b4.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/cc7d47f25b44