1866438 - Create a nimbus rollout for the highlighting feature

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, task)

Description

[Marco Castelluccio [:marco]]

We need to do something similar to bug 1859365 for the highlighting feature.

Comment 1

[Marco Castelluccio [:marco]]

Barret, do we need to implement an exposure event to identify the population who's actually opening PDFs or can we get this info some other way?

Comment 2

[Beth Rennie [:beth] (she/her)]

I believe you would have to implement an exposure event -- I don't know if we have any telemetry regarding opening PDFs.

Comment 3

[Calixte Denizet (:calixte)]

Attachment: Bug 1866438 - Register pdfjs.enableHighlightEditor with Nimbus r=marco,barret

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4752ecef06b0
Register pdfjs.enableHighlightEditor with Nimbus r=marco,barret

Comment 5

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/4752ecef06b0

Comment 6

[Calixte Denizet (:calixte)]

Comment on attachment 9382630 [details]
Bug 1866438 - Register pdfjs.enableHighlightEditor with Nimbus r=marco,barret

Beta/Release Uplift Approval Request

• User impact if declined: No experiment for the highlight feature in pdf.js.
• 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: Bug 1866437, Bug 1883050
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Very small change and pdf.js is self contained.
• String changes made/needed:
• Is Android affected?: No

Comment 7

[Dianna Smith [:diannaS]]

Comment on attachment 9382630 [details]
Bug 1866438 - Register pdfjs.enableHighlightEditor with Nimbus r=marco,barret

Approved for 124.0b7

Comment 8

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-beta/rev/3796d15b6aab