diff options
| author | vitalybuka@chromium.org <vitalybuka@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-11 19:11:40 +0000 |
|---|---|---|
| committer | vitalybuka@chromium.org <vitalybuka@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-11 19:11:40 +0000 |
| commit | 263c351a232794b3ae826973a71e5a074fe547ea (patch) | |
| tree | b8a102af95e04192c3a5911fd80a4d6dbd532d87 /printing/printing_context_win.cc | |
| parent | 3a2d2789e42736bcc0ff55d9a0b9a1d27ef2a2d3 (diff) | |
| download | chromium_src-263c351a232794b3ae826973a71e5a074fe547ea.zip chromium_src-263c351a232794b3ae826973a71e5a074fe547ea.tar.gz chromium_src-263c351a232794b3ae826973a71e5a074fe547ea.tar.bz2 | |
Don't close handles returned by ::OpenPrinter with error.
BUG=248061
Review URL: https://chromiumcodereview.appspot.com/15798023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205600 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'printing/printing_context_win.cc')
| -rw-r--r-- | printing/printing_context_win.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/printing/printing_context_win.cc b/printing/printing_context_win.cc index b5a57cf..46aa98f 100644 --- a/printing/printing_context_win.cc +++ b/printing/printing_context_win.cc @@ -371,7 +371,7 @@ PrintingContext::Result PrintingContextWin::UpdatePrinterSettings( ScopedPrinterHandle printer; LPWSTR device_name_wide = const_cast<wchar_t*>(device_name.c_str()); - if (!OpenPrinter(device_name_wide, printer.Receive(), NULL)) + if (!printer.OpenPrinter(device_name_wide)) return OnError(); // Make printer changes local to Chrome. @@ -441,9 +441,9 @@ PrintingContext::Result PrintingContextWin::InitWithSettings( // TODO(maruel): settings_.ToDEVMODE() ScopedPrinterHandle printer; - if (!OpenPrinter(const_cast<wchar_t*>(settings_.device_name().c_str()), - printer.Receive(), NULL)) + if (!printer.OpenPrinter(settings_.device_name().c_str())) { return FAILED; + } Result status = OK; |