Ensure that printer job handler is stopped and task_in_progress_ is reset in OnAuthError

chrome/service/cloud_print/printer_job_handler.cc

 // Copyright (c) 2012 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/printer_job_handler.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/file_util.h"
 #include "base/json/json_reader.h"
@@ skipping 160 matching lines @@
       ret = true;
       break;
     }
   }
   return ret;
 }
 
 void PrinterJobHandler::OnAuthError() {
   if (delegate_)
     delegate_->OnAuthError();
+  MessageLoop::current()->PostTask(

[gene, 2012/07/25 18:45:43]

Move to the top of the function?
It probably does not matter here, but it is better to schedule stop first.

[Alex Yu, 2012/07/25 18:51:44]

Done.

+      FROM_HERE, base::Bind(&PrinterJobHandler::Stop, this))
 }
 
 void PrinterJobHandler::OnPrinterDeleted() {
   if (delegate_)
     delegate_->OnPrinterDeleted(printer_info_cloud_.printer_id);
 }
 
 void PrinterJobHandler::OnPrinterChanged() {
   printer_update_pending_ = true;
   if (!task_in_progress_) {
@@ skipping 486 matching lines @@
   if (!job_spooler_ || !job_spooler_->Spool(job_details.print_ticket_,
                                             job_details.print_data_file_path_,
                                             job_details.print_data_mime_type_,
                                             printer_name,
                                             job_details.job_title_,
                                             job_details.tags_,
                                             this)) {
     OnJobSpoolFailed();
   }
 }