Printing Backend: Check the results from a DocumentProperties() call.

printing/backend/print_backend_win.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "printing/backend/print_backend.h"
 
 #include <objidl.h>
 #include <winspool.h>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 119 matching lines @@
                                    &printer_info->printer_capabilities);
       DCHECK(SUCCEEDED(hr));
       printer_info->caps_mime_type = "text/xml";
     }
     // TODO(sanjeevr): Add ScopedPrinterHandle
     HANDLE printer_handle = NULL;
     OpenPrinter(const_cast<LPTSTR>(printer_name_wide.c_str()), &printer_handle,
                 NULL);
     DCHECK(printer_handle);
     if (printer_handle) {
-      DWORD devmode_size = DocumentProperties(
+      LONG devmode_size = DocumentProperties(
           NULL, printer_handle, const_cast<LPTSTR>(printer_name_wide.c_str()),
           NULL, NULL, 0);
-      DCHECK_NE(0U, devmode_size);
+      if (devmode_size <= 0)
+        return false;
       scoped_ptr<BYTE> devmode_out_buffer(new BYTE[devmode_size]);
       DEVMODE* devmode_out =
           reinterpret_cast<DEVMODE*>(devmode_out_buffer.get());
       DocumentProperties(
           NULL, printer_handle, const_cast<LPTSTR>(printer_name_wide.c_str()),
           devmode_out, NULL, DM_OUT_BUFFER);
       base::win::ScopedComPtr<IStream> printer_defaults_stream;
       hr = CreateStreamOnHGlobal(NULL, TRUE,
                                  printer_defaults_stream.Receive());
       DCHECK(SUCCEEDED(hr));
@@ skipping 30 matching lines @@
   }
   return ret;
 }
 
 scoped_refptr<PrintBackend> PrintBackend::CreateInstance(
     const base::DictionaryValue* print_backend_settings) {
   return new PrintBackendWin;
 }
 
 }  // namespace printing