Closed Bug 1617204 Opened 6 years ago Closed 1 year ago

We overcount PRINT_PREVIEW_SIMPLIFY_PAGE_UNAVAILABLE_COUNT

Categories

(Toolkit :: Printing, defect, P2)

defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: jwatt, Assigned: jwatt)

References

Details

We collect some print preview telemetry, including:

  • PRINT_PREVIEW_OPENED_COUNT
  • PRINT_PREVIEW_SIMPLIFY_PAGE_UNAVAILABLE_COUNT
  • PRINT_PREVIEW_SIMPLIFY_PAGE_OPENED_COUNT

You might expect that PRINT_PREVIEW_SIMPLIFY_PAGE_UNAVAILABLE_COUNT would be a subset of PRINT_PREVIEW_OPENED_COUNT, but that's not the case.

The method that we use to initiate print preview is the same as the method we use to re-layout the print preview for settings changes (portrate -> landscape, for example). Specifically, we call PrintUtils.printPreview(). In this function we take care not to increment PRINT_PREVIEW_OPENED_COUNT when printPreview() is invoked additional times to apply settings changes, but we do not do this for PRINT_PREVIEW_SIMPLIFY_PAGE_UNAVAILABLE_COUNT.

While fixing this issue, it's probably worth considering whether we want to only increment PRINT_PREVIEW_SIMPLIFY_PAGE_OPENED_COUNT once per print preview session too. I think it would be more useful if we did.

Status: NEW → ASSIGNED
Severity: normal → S3

This telemetry was since removed.

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.