1664489 - Print preview telemetry for the legacy UI is broken

Status: RESOLVED WONTFIX

(Core :: Printing: Setup, defect, P3)

Description

[Jonathan Watt [:jwatt]]

Prior to bug 1659432 we used to use a single nsPrintJob, but now we replace it every time that the print settings are changed, and when the user actually hits "Print". Unfortunately nsPrintJob::mCreatedForPrintPreview is not propagated to the new nsPrintJob, so in nsPrintJob::DoCommonPrint we will misattribute PRINTING_DIALOG_OPENED_VIA_PREVIEW and PRINTING_DIALOG_VIA_PREVIEW_CANCELLED to PRINTING_DIALOG_OPENED_WITHOUT_PREVIEW and PRINTING_DIALOG_WITHOUT_PREVIEW_CANCELLED respectively.

Comment 1

[Jonathan Watt [:jwatt]]

[Tracking Requested - why for this release]: Printing telemetry broken.

Comment 2

[Jonathan Watt [:jwatt]]

Attachment: Bug 1664489. Fix print preview telemetry for the legacy UI. r=emilio

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1659432

Comment 4

[Ryan VanderMeulen [:RyanVM]]

I'm building the RC for 81 today. Does this mean I'm going to need a respin later in the week?

Comment 5

[Jonathan Watt [:jwatt]]

The fix for this has turned out to be too risky for so late in Beta. I've discussed the issue with Martin and I won't be requesting uplift. Thanks, Ryan.

Comment 6

[Blake Winton (:bwinton) (:☕️)]

(Moving to v82 in the whiteboard, since we won't be uplifting it.)

Comment 7

[Sean Voisen (:svoisen)]

Deprioritizing since we don't have current plans to fix this.

Comment 8

[Sean Voisen (:svoisen)]

We don't really rely on this telemetry anymore, so reassessing as S3.