1912500 - Implement the telemetry for the new alt text flow

Status: VERIFIED FIXED

(Firefox :: PDF Viewer, enhancement, P1)

Description

[Calixte Denizet (:calixte)]

In bug 1909325, I just added the definitions in the mretrics file but I still have to add the code to use them.

Comment 1

[BMO Github Automation]

Attachment: [mozilla/pdf.js] [Editor] Add the telemetry for the new alt text (bug 1912500) (#18589)

Comment 2

[Calixte Denizet (:calixte)]

Attachment: Bug 1912500 - Add the code for the new alt-text flow telemetry r=tarek

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c57f22e881f9
Add the code for the new alt-text flow telemetry r=tarek

Comment 4

[Calixte Denizet (:calixte)]

Attachment: Bug 1912500 - Add the code for the new alt-text flow telemetry

Original Revision: https://phabricator.services.mozilla.com/D218975

Depends on D219263

Comment 5

[Phabricator Automation]

beta Uplift Approval Request

• User impact if declined: Low
• Code covered by automated testing: yes
• Fix verified in Nightly: no
• Needs manual QE test: yes
• Steps to reproduce for manual QE testing: Check that the telemetry is correctly implemented
• Risk associated with taking this patch: low
• Explanation of risk level: the new alt-text feature is behind a pref and will be only an experiment in 130
• String changes made/needed: Yes in D219261 there are some string changes.
• Is Android affected?: no

Comment 6

[pstanciu]

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

Comment 7

[Pulsebot]

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

Comment 8

[Pulsebot]

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

Comment 9

[Rares Doghi, Desktop QA]

Verified as fixed in our latest Nightly and Beta 130.0b6 .