Printing: Add ability to get the default printer from the print backend.

Printing: Add ability to get the default printer from the print backend.

BUG=84126
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=87187

[Lei Zhang, 2011-05-28 00:57:37]

It turns out in many cases, enumerating the printers takes longer than the time
it takes to preview. So if we can get the default printer, we can generate a
preview for that printer in parallel with printer enumeration.

This is just the change to the PrintBackend.

[vandebo (ex-Chrome), 2011-05-28 18:21:39]

LGTM

[vandebo (ex-Chrome), 2011-05-28 18:22:01]

Oops, here are my nits.  LGTM

http://codereview.chromium.org/7087011/diff/5/printing/backend/print_backend_...
File printing/backend/print_backend_win.cc (right):

http://codereview.chromium.org/7087011/diff/5/printing/backend/print_backend_...
printing/backend/print_backend_win.cc:91: // Getting the name of the default
printer.
nit: comment is unnecessary in this context.

http://codereview.chromium.org/7087011/diff/5/printing/backend/print_backend_...
printing/backend/print_backend_win.cc:94: BOOL default_printer_exists =
::GetDefaultPrinter(
nit: you can ditch the bool variable and just put the call in the if
conditional.

[Lei Zhang, 2011-05-28 20:50:18]

http://codereview.chromium.org/7087011/diff/5/printing/backend/print_backend_...
File printing/backend/print_backend_win.cc (right):

http://codereview.chromium.org/7087011/diff/5/printing/backend/print_backend_...
printing/backend/print_backend_win.cc:91: // Getting the name of the default
printer.
On 2011/05/28 18:22:01, vandebo wrote:
> nit: comment is unnecessary in this context.

Done.

http://codereview.chromium.org/7087011/diff/5/printing/backend/print_backend_...
printing/backend/print_backend_win.cc:94: BOOL default_printer_exists =
::GetDefaultPrinter(
On 2011/05/28 18:22:01, vandebo wrote:
> nit: you can ditch the bool variable and just put the call in the if
> conditional.

Done.

[vandebo (ex-Chrome), 2011-05-28 21:19:52]

LG

[commit-bot: I haz the power, 2011-05-28 21:50:30]

Change committed as 87187