Cleanup jobs which have failed.

chrome/browser/chromeos/printing/cups_print_job.h

Index: chrome/browser/chromeos/printing/cups_print_job.h
diff --git a/chrome/browser/chromeos/printing/cups_print_job.h b/chrome/browser/chromeos/printing/cups_print_job.h
index 09be00de135cafd0b8c5b4af56c3f69af14efcf1..997989fa715f167634415e6e6534651bd33a3751 100644
--- a/chrome/browser/chromeos/printing/cups_print_job.h
+++ b/chrome/browser/chromeos/printing/cups_print_job.h
@@ -52,6 +52,7 @@ class CupsPrintJob {
   const std::string& document_title() const { return document_title_; }
   int total_page_number() const { return total_page_number_; }
   int printed_page_number() const { return printed_page_number_; }
+  bool expired() const { return expired_; }
   State state() const { return state_; }
   ErrorCode error_code() const { return error_code_; }
 
@@ -59,6 +60,7 @@ class CupsPrintJob {
   void set_printed_page_number(int page_number) {
     printed_page_number_ = page_number;
   }
+  void set_expired(bool expired) { expired_ = expired; }
   void set_state(State state) { state_ = state; }
   void set_error_code(ErrorCode error_code) { error_code_ = error_code; }
 
@@ -73,6 +75,9 @@ class CupsPrintJob {
   int total_page_number_ = 0;
   int printed_page_number_ = 0;
 
+  // True if the job has expired due to timeout.
+  bool expired_ = false;

[xdai1, 2017/03/16 19:56:16]

Maybe we can merge this variable to ErrorCode?  

[skau, 2017/03/16 22:14:36]

We could.  But I think it'll be easier to maintain with separate fields. 
ErrorCode indicates what the problem is.  expired indicates that the job needs
to be removed from the print queue.

[xdai1, 2017/03/16 22:56:14]

Acknowledged.

+
   State state_ = State::STATE_NONE;
   ErrorCode error_code_ = ErrorCode::NO_ERROR;