1658300 - Implement a Promise returning nsIPrinterList.getNamedOrDefaultPrinter(lastUsedPrinterName)

Status: RESOLVED FIXED

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

Description

[Jonathan Watt [:jwatt]]

Right now the frontend code has to wait for the entire list of printers to be fetched in order to iterate over that list and check whether the last used printer is still available. Fetching the entire list can potentially take a long time. It seems likely that it may be much better if the frontend code could do:

  let lastUsedPrinterName = printSettingsService.lastUsedPrinterName;
  let printer = await printerList.getNamedOrDefaultPrinter(lastUsedPrinterName);
  let printerName = printer.name;
  // ...

That way there will be less delay before it can get hold of the nsIPrinter that it should setup print preview with, and it can get on and message the platform code to start creating the print preview document sooner.

Comment 1

[Emily McDonough [:alaskanemily]]

Attachment: Bug 1658300 - Implement a promise-based nsIPrinterList::getNamedOrDefaultPrinter. r=jwatt,emilio

Also fix the odering of the methods in nsPrinterListCUPS while I'm messing
with it.

Comment 2

[Jonathan Kew [:jfkthame]]

Attachment: Bug 1658300 - Update nsPrinterListWin to implement internal SystemDefaultPrinterName() instead of the IDL Get* method. r=bobowen

Comment 3

[Jonathan Kew [:jfkthame]]

Attachment: Bug 1658300 - Implement NamedPrinter() for nsPrinterListWin, and remove the base implementation. r=bobowen

Depends on D87449

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6eb8ce9c2299
Implement a promise-based nsIPrinterList::getNamedOrDefaultPrinter. r=jwatt,emilio
https://hg.mozilla.org/integration/autoland/rev/5f947346134e
Implement NamedPrinter() for nsPrinterListWin, and remove the base implementation. r=bobowen

Comment 5

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/6eb8ce9c2299
https://hg.mozilla.org/mozilla-central/rev/5f947346134e