Closed
Bug 1803654
Opened 3 years ago
Closed 3 years ago
Update PDF.js annotation Feature Callout messages to include both treatment branches
Categories
(Firefox :: Messaging System, task, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox109 | --- | verified |
People
(Reporter: mviar, Assigned: mviar)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Update the PDF.js annotation messages in FeatureCalloutMessages.jsm to include four messages reflecting the two dual screen messages outlined in the experiment brief. We'll deploy the experiment to set the value of the pdf js tour pref to the screen id for each respective branch.
Assignee | ||
Comment 1•3 years ago
|
||
Updated•3 years ago
|
Attachment #9306301 -
Attachment description: WIP: Bug 1803654 - Update PDF.js annotation Feature Callout messages to include both treatment branches → Bug 1803654 - Update PDF.js annotation Feature Callout messages to include both treatment branches
Pushed by mviar@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/988063d03a9e
Update PDF.js annotation Feature Callout messages to include both treatment branches r=jprickett
Comment 3•3 years ago
|
||
Backed out for causing Talos crashes on pdfpaint.
Failure log: https://treeherder.mozilla.org/logviewer?job_id=398512181&repo=autoland&lineNumber=6869
Backout link: https://hg.mozilla.org/integration/autoland/rev/492c6608613f83b6ebde6a9d1c76f9565107c7b5
Flags: needinfo?(mviar)
Pushed by csabou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ec0bbb314eaf
Update PDF.js annotation Feature Callout messages to include both treatment branches r=jprickett
Comment 5•3 years ago
|
||
The crashes turned out to be from Bug 1803604 so this was relanded. ^
Flags: needinfo?(mviar)
Comment 6•3 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Comment 8•3 years ago
|
||
I'm marking this task as verified based on the testing report sent for the "Callout: Pdf annotations for Fx109" bugwork.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•