Closed Bug 1740630 Opened 23 days ago Closed 15 days ago

Printing selection from parent process browsers is broken and disabled

Categories

(Toolkit :: Printing, defect)

defect

Tracking

()

RESOLVED FIXED
96 Branch
Tracking Status
firefox96 --- fixed

People

(Reporter: darktrojan, Assigned: darktrojan)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

While looking into selection printing in Thunderbird (bug 1731183) I discovered that printing the selection from the parent process is disabled because it doesn't work properly. It doesn't work properly because this code is trying to get the selection from the print settings window instead of the window being printed. That can be fixed.

Blocks: 1731183
Attachment #9250305 - Attachment description: Bug 1740630 - Fix and enable printing the selection from a parent process browser. r=mstriemer → Bug 1740630 - Fix and enable printing the selection from a parent process browser. r=emilio
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/integration/autoland/rev/e9cb0621e5a1
Fix and enable printing the selection from a parent process browser. r=emilio
Status: ASSIGNED → RESOLVED
Closed: 15 days ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch

Comment on attachment 9250305 [details]
Bug 1740630 - Fix and enable printing the selection from a parent process browser. r=mstriemer

Beta/Release Uplift Approval Request

  • User impact if declined: Firefox users are largely unaffected, but Thunderbird users are unable to print only a selection. I'd like this to make it to Thunderbird 91.4.0 if possible.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky):
  • String changes made/needed:
Attachment #9250305 - Attachment description: Bug 1740630 - Fix and enable printing the selection from a parent process browser. r=emilio → Bug 1740630 - Fix and enable printing the selection from a parent process browser. r=mstriemer
Attachment #9250305 - Flags: approval-mozilla-beta?

Comment on attachment 9250305 [details]
Bug 1740630 - Fix and enable printing the selection from a parent process browser. r=mstriemer

It seems that you want it on the ESR branch, not Firefox 95 beta branch.

Attachment #9250305 - Flags: approval-mozilla-beta? → approval-mozilla-esr91?

Well yes, but is going to beta first not a requirement?

Flags: needinfo?(pascalc)

(In reply to Geoff Lankow (:darktrojan) from comment #6)

Well yes, but is going to beta first not a requirement?

I am not comfortable taking this patch into beta because this landed recently on mozilla-central and our RC week is next week, also although this uplift says that this was verified in nightly, I don't think it was done by QA. Historically, changes in our printing code tends to lead to regressions so I'd prefer to have this patch ride the train and uplift in ESR 91.5.0 or 91.6.0, the time for potential regressions to be reported. Given that unlike for Firefox, this bug has a high impact on Thunderbird, I understand that this is something you'd like to see in your next point release, I guess that you can cherry-pick this patch for TB91.4.0 until we land it on ESR though?

Flags: needinfo?(pascalc) → needinfo?(geoff)

We don't have an ESR branch to cherry-pick it to. We use mozilla-esr91. Whatever, it can wait a cycle, it's been several months already, a few more weeks won't hurt.

Flags: needinfo?(geoff)

Good call. It can wait a cycle

You need to log in before you can comment on or make changes to this bug.