Report print jobs that are aborted by CUPS as errors.

chrome/browser/chromeos/printing/cups_print_job_notification.h

Index: chrome/browser/chromeos/printing/cups_print_job_notification.h
diff --git a/chrome/browser/chromeos/printing/cups_print_job_notification.h b/chrome/browser/chromeos/printing/cups_print_job_notification.h
index a5f211d583fc7aa657161f9b2267293ce76e666e..c55486390ca7e775469a3cc952d3de05af6c2bcb 100644
--- a/chrome/browser/chromeos/printing/cups_print_job_notification.h
+++ b/chrome/browser/chromeos/printing/cups_print_job_notification.h
@@ -5,6 +5,10 @@
 #ifndef CHROME_BROWSER_CHROMEOS_PRINTING_CUPS_PRINT_JOB_NOTIFICATION_H_
 #define CHROME_BROWSER_CHROMEOS_PRINTING_CUPS_PRINT_JOB_NOTIFICATION_H_
 
+#include <memory>
+#include <string>
+#include <vector>

[xdai1, 2017/02/24 23:31:52]

Just curious: Normally I don't proactively add more head files if the compiler
doesn't complain. Any guide rules about when should add more head files? 

[skau, 2017/02/25 00:16:06]

It shows up as a linter warning when I run 'git cl lint'.  It also improves
build stability in the case where you're getting <vector> from  a different
header file then that header removes it.  The build will break in an unexpected
way.

+
 #include "chrome/browser/notifications/notification.h"
 #include "chrome/browser/notifications/notification_delegate.h"
 
@@ -59,6 +63,10 @@ class CupsPrintJobNotification {
   // and only show the print job done or failed notification.
   bool closed_in_middle_ = false;
 
+  // If this is true, the user cancelled the job using the cancel button and
+  // should not be notified of events.
+  bool cancelled_by_user_ = false;
+
   // Maintains a list of button actions according to the print job's current
   // status.
   std::unique_ptr<std::vector<ButtonCommand>> button_commands_;