Closed Bug 1659856 Opened 4 years ago Closed 4 years ago

Support nsIPrintSetting.duplex on macOS and Windows

Categories

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

defect

Tracking

()

RESOLVED FIXED
82 Branch
Tracking Status
firefox81 --- fixed
firefox82 --- fixed

People

(Reporter: jwatt, Assigned: jfkthame)

References

(Blocks 1 open bug)

Details

(Whiteboard: [print2020_v81])

Attachments

(4 files, 1 obsolete file)

It looks like only the GTK code uses nsIPrintSetting.duplex. As far as I can tell, we don't do anything if this is set on macOS or Windows. We'll need to fix that in order for the new printing UI to implement the "Print on both sides" option and have it actually work.

I've attached patches that I think ought to make this work for Windows and macOS, but they are untested -- I don't have access to a printer with duplexing options, so it's hard to confirm whether they result in the expected behavior!

Erik, Emily: are either of you able to test this at all, by any chance (for either platform)? If so, that'd be awesome.

Flags: needinfo?(enordin)
Flags: needinfo?(emcdonough)

I read my printer's manual and it says that it supports Duplex printing for Windows, but not on macOS, unfortunately.

Sean, you said that you have a pretty fancy printer, right? Do you think you could test the duplex printing on macOS here?

Flags: needinfo?(enordin) → needinfo?(svoisen)

Sorry, no fancy printer here. I used to have access to a color laser that did duplex, but no more.

Flags: needinfo?(svoisen)

OK, I suggest we just get this landed on the platform side, even though it's currently untested, so that as soon as it's exposed in the front-end anyone (QA?) with a duplex-capable printer can try it out and confirm if all works as expected.

Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Attachment #9171544 - Attachment description: Bug 1659856 - patch 2 - Handle duplex setting in nsPrintSettingsWin. → Bug 1659856 - patch 2 - Handle duplex setting in nsPrintSettingsWin. r=jwatt
Attachment #9171545 - Attachment description: Bug 1659856 - patch 3 - Handle duplex setting in nsPrintSettingsX for macOS. → Bug 1659856 - patch 3 - Handle duplex setting in nsPrintSettingsX for macOS. r=jwatt
Pushed by jkew@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f0090d86ffb5 patch 1 - Add IDL constants for duplexing options to nsIPrintSettings. r=jwatt https://hg.mozilla.org/integration/autoland/rev/e0b318464ca0 patch 2 - Handle duplex setting in nsPrintSettingsWin. r=jwatt https://hg.mozilla.org/integration/autoland/rev/b61a789151df patch 3 - Handle duplex setting in nsPrintSettingsX for macOS. r=jwatt
Attachment #9172158 - Attachment is obsolete: true
Pushed by jkew@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0451aa9d80c7 patch 1 - Add IDL constants for duplexing options to nsIPrintSettings. r=jwatt https://hg.mozilla.org/integration/autoland/rev/8387257bb40e patch 2 - Handle duplex setting in nsPrintSettingsWin. r=jwatt https://hg.mozilla.org/integration/autoland/rev/5d6896aa46ad patch 3 - Handle duplex setting in nsPrintSettingsX for macOS. r=jwatt
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
Flags: needinfo?(jfkthame)

Reminder to request Beta approval.

Flags: needinfo?(jfkthame)

Do we need to uplift this? I thought duplex was targeted for 82.

Flags: needinfo?(jfkthame) → needinfo?(jwatt)

I guess not. It should be harmless to uplift though since it's currently unused code, and uplifting has the benefit that it makes it less likely that other printing patches that we want to uplift will encounter conflicts. I'd lean towards uplift, but up to you.

Flags: needinfo?(jwatt)

Comment on attachment 9171543 [details]
Bug 1659856 - patch 1 - Add IDL constants for duplexing options to nsIPrintSettings.

Beta/Release Uplift Approval Request

  • User impact if declined: No direct user impact (unused code), but may make other uplifts more difficult over the next few weeks as there are likely to be other patches in nearby code that are needed for the Firefox 81 new printing UI.
  • 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: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Currently-unused code (preparing platform for a feature not yet exposed in the printing UI).
  • String changes made/needed:
Attachment #9171543 - Flags: approval-mozilla-beta?
Attachment #9171544 - Flags: approval-mozilla-beta?
Attachment #9171545 - Flags: approval-mozilla-beta?

Comment on attachment 9172659 [details]
Bug 1659856 - followup - Also handle duplex field in nsPrintSettingsWin::InitWithInitializer. r=bobowen

(See comment 16 for uplift form.)

Attachment #9172659 - Flags: approval-mozilla-beta?
Pushed by jkew@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0312e30104cc followup - Also handle duplex field in nsPrintSettingsWin::InitWithInitializer. r=bobowen

Comment on attachment 9171543 [details]
Bug 1659856 - patch 1 - Add IDL constants for duplexing options to nsIPrintSettings.

Approved for 81.0b4.

Attachment #9171543 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Attachment #9171544 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Attachment #9171545 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Attachment #9172659 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Whiteboard: [print2020_v82] → [print2020_v81]
Flags: needinfo?(emcdonough)
Regressed by: 1689412
No longer regressed by: 1689412
Regressions: 1689412
No longer regressions: 1689412
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: