Added PrinterDriverInfo and PrintBackend::GetPrinterDriverInfo for windows.

Added PrinterDriverInfo and PrintBackend::GetPrinterDriverInfo for windows.

BUG=108194
TEST=none


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

[Albert Bodenhamer, 2012-03-02 21:14:03]

http://codereview.chromium.org/9516010/diff/15002/base/win/scoped_handle.h
File base/win/scoped_handle.h (right):

http://codereview.chromium.org/9516010/diff/15002/base/win/scoped_handle.h#ne...
base/win/scoped_handle.h:1: // Copyright (c) 2010 The Chromium Authors. All
rights reserved.
Update copyright

http://codereview.chromium.org/9516010/diff/15002/base/win/scoped_handle.h#ne...
base/win/scoped_handle.h:34: class GenericScopedHandle {
If you need to change ScopedHandle it's probably worth breaking that into a
separate CL.

[Vitaly Buka (NO REVIEWS), 2012-03-02 22:45:25]

Sorry, but I already pushed unrelated parts of CL into CQ.
I will remove changes from here and ask for review later.

BTW. Looks like copyright was updated by CQ automatically
See here. I didn't change in CL. 
http://codereview.chromium.org/9480026/diff/15009/base/win/scoped_handle.h

But version in SVN is correct
http://src.chromium.org/viewvc/chrome/trunk/src/base/win/scoped_handle.h?view...

On 2012/03/02 21:14:03, Albert Bodenhamer wrote:
> http://codereview.chromium.org/9516010/diff/15002/base/win/scoped_handle.h
> File base/win/scoped_handle.h (right):
> 
>
http://codereview.chromium.org/9516010/diff/15002/base/win/scoped_handle.h#ne...
> base/win/scoped_handle.h:1: // Copyright (c) 2010 The Chromium Authors. All
> rights reserved.
> Update copyright
> 
>
http://codereview.chromium.org/9516010/diff/15002/base/win/scoped_handle.h#ne...
> base/win/scoped_handle.h:34: class GenericScopedHandle {
> If you need to change ScopedHandle it's probably worth breaking that into a
> separate CL.

[Vitaly Buka (NO REVIEWS), 2012-03-03 00:30:54]

Please review now.

[Albert Bodenhamer, 2012-03-05 17:57:53]

lgtm

http://codereview.chromium.org/9516010/diff/12009/printing/backend/print_back...
File printing/backend/print_backend_cups.cc (right):

http://codereview.chromium.org/9516010/diff/12009/printing/backend/print_back...
printing/backend/print_backend_cups.cc:233: // TODO(vitalybuka): implement this.
Reference the bug number in the todo.

[Vitaly Buka (NO REVIEWS), 2012-03-05 19:24:50]

http://codereview.chromium.org/9516010/diff/12009/printing/backend/print_back...
File printing/backend/print_backend_cups.cc (right):

http://codereview.chromium.org/9516010/diff/12009/printing/backend/print_back...
printing/backend/print_backend_cups.cc:233: // TODO(vitalybuka): implement this.
On 2012/03/05 17:57:53, Albert Bodenhamer wrote:
> Reference the bug number in the todo.

Done.

[commit-bot: I haz the power, 2012-03-05 19:25:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vitalybuka@chromium.org/9516010/22001

[commit-bot: I haz the power, 2012-03-05 23:17:28]

Change committed as 125020