1658220 - Bug 1657363 breaks printer selection on windows because "last used printer" mechanism is broken.

Status: RESOLVED FIXED

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

Description

[Emilio Cobos Álvarez [:emilio]]

I gave bug 1657363 a spin on windows and it regresses some behavior. Turns out that the default printer mechanism is busted with the new UI, but it kinda happened to work due to two things:

• When there's no previous printer we used to fall back to system default before bug 1657363.
• Windows changes the system default printer each time you print, so this worked just as you'd expect.

Yay.

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1658220 - When building the printer list, select the system default if we don't find the last used printer. r=mstriemer,jwatt

This effectively restores the behavior on windows, though it still
doesn't fix the "save last used printer" functionality. This is the only
caller affected because on the others we fall back to the system default
if there's no printer name specified...

The map() is not the prettier, let me know if you have something cleaner
in mind.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3d44841a5c38
Workaround old gcc which bogusly warns about hiding overloaded functions with different arguments by renaming a function.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e7cd27996ce2
When building the printer list, select the system default if we don't find the last used printer. r=jwatt

Comment 4

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/3d44841a5c38
https://hg.mozilla.org/mozilla-central/rev/e7cd27996ce2

Comment 5

[Emilio Cobos Álvarez [:emilio]]

Let's call this fixed, as it fixed the regression. Bob filed bug 1659122 independently to track the underlying "saving the printer is broken", so let's fix that there.

Comment 6

[Max Jack]

I have also printin problem from my printer . i have read a blog from a <a href="https://123printersupport.com/">printer support</a> website but i am unable to find that.