Fixed a corner case issue with printing XPS documents to the Microsoft XPS Do...

chrome/service/cloud_print/print_system_win.cc

 // Copyright (c) 2010 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 "chrome/service/cloud_print/print_system.h"
 
 #include <objidl.h>
 #include <winspool.h>
 #if defined(_WIN32_WINNT)
 #undef _WIN32_WINNT
@@ skipping 446 matching lines @@
         last_page_printed_ = highest_rendered_page_number;
         if (done_printing)
           PrintJobDone();
         else
           RenderNextPDFPages();
       }
 
       // base::win::ObjectWatcher::Delegate inplementation.
       virtual void OnObjectSignaled(HANDLE object) {
         DCHECK(xps_print_job_);
+        DCHECK(object == job_progress_event_.Get());
+        ResetEvent(job_progress_event_.Get());
         if (!delegate_)
           return;
         XPS_JOB_STATUS job_status = {0};
         xps_print_job_->GetJobStatus(&job_status);
         bool done = false;
         if ((job_status.completion == XPS_JOB_CANCELLED) ||
             (job_status.completion == XPS_JOB_FAILED)) {
           delegate_->OnJobSpoolFailed();
           done = true;
-        } else if (job_status.jobId) {
+        } else if (job_status.jobId ||
+                   (job_status.completion == XPS_JOB_COMPLETED)) {
+          // Note: In the case of the XPS document being printed to the
+          // Microsoft XPS Document Writer, it seems to skip spooling the job
+          // and goes to the completed state without ever assigning a job id.
           delegate_->OnJobSpoolSucceeded(job_status.jobId);
           done = true;
         } else {
-          ResetEvent(job_progress_event_.Get());
           job_progress_watcher_.StopWatching();
           job_progress_watcher_.StartWatching(job_progress_event_.Get(), this);
         }
         if (done && should_couninit_) {
           CoUninitialize();
           should_couninit_ = false;
         }
       }
 
       virtual void OnRenderPDFPagesToMetafileFailed() {
@@ skipping 282 matching lines @@
         NULL,
         kPTJobScope,
         result_ticket_stream.get(),
         error.Receive()));
     printing::XPSModule::CloseProvider(provider);
   }
   return ret;
 }
 
 bool PrintSystemWin::GetJobDetails(const std::string& printer_name,
-                                       PlatformJobId job_id,
-                                       PrintJobDetails *job_details) {
+                                   PlatformJobId job_id,
+                                   PrintJobDetails *job_details) {
   DCHECK(job_details);
   HANDLE printer_handle = NULL;
   std::wstring printer_name_wide = UTF8ToWide(printer_name);
   OpenPrinter(const_cast<LPTSTR>(printer_name_wide.c_str()), &printer_handle,
               NULL);
   DCHECK(printer_handle);
   bool ret = false;
   if (printer_handle) {
     DWORD bytes_needed = 0;
     GetJob(printer_handle, job_id, 1, NULL, 0, &bytes_needed);
@@ skipping 64 matching lines @@
   RpcStringFree(reinterpret_cast<RPC_WSTR *>(&proxy_id_as_string));
   return ret;
 }
 
 scoped_refptr<PrintSystem> PrintSystem::CreateInstance(
     const DictionaryValue* print_system_settings) {
   return new PrintSystemWin;
 }
 
 }  // namespace cloud_print