Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1313)

Unified Diff: chrome/browser/chromeos/printing/cups_print_job_notification_manager.cc

Issue 2708233006: Report print jobs that are aborted by CUPS as errors. (Closed)
Patch Set: ready Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/printing/cups_print_job_notification_manager.cc
diff --git a/chrome/browser/chromeos/printing/cups_print_job_notification_manager.cc b/chrome/browser/chromeos/printing/cups_print_job_notification_manager.cc
index b7a6636416432e87d7592ee25f6a8d7d0eb9741f..ea97d265ab695dda8876aeb6df9e506c7705d9f9 100644
--- a/chrome/browser/chromeos/printing/cups_print_job_notification_manager.cc
+++ b/chrome/browser/chromeos/printing/cups_print_job_notification_manager.cc
@@ -60,4 +60,11 @@ void CupsPrintJobNotificationManager::OnPrintJobError(CupsPrintJob* job) {
notification_map_[job]->OnPrintJobStatusUpdated();
}
+void CupsPrintJobNotificationManager::OnPrintJobCancelled(CupsPrintJob* job) {
+ auto notification = notification_map_.find(job);
xdai1 2017/02/24 19:17:00 Why not use the same format in the other functions
skau 2017/02/24 23:09:34 I've changed it. I had thought notifications were
+ if (notification != notification_map_.end()) {
+ notification->second->OnPrintJobStatusUpdated();
+ }
+}
+
} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698