[CUPS] Implement the Print Job notification.

chrome/browser/chromeos/printing/cups_print_job_notification_manager.h

Index: chrome/browser/chromeos/printing/cups_print_job_notification_manager.h
diff --git a/chrome/browser/chromeos/printing/cups_print_job_notification_manager.h b/chrome/browser/chromeos/printing/cups_print_job_notification_manager.h
new file mode 100644
index 0000000000000000000000000000000000000000..bad14c895cabc268c7a378f00d773a8146a614d8
--- /dev/null
+++ b/chrome/browser/chromeos/printing/cups_print_job_notification_manager.h
@@ -0,0 +1,50 @@
+// Copyright 2016 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.
+
+#ifndef CHROME_BROWSER_CHROMEOS_PRINTING_CUPS_PRINT_JOB_NOTIFICATION_MANAGER_H_
+#define CHROME_BROWSER_CHROMEOS_PRINTING_CUPS_PRINT_JOB_NOTIFICATION_MANAGER_H_
+
+#include <memory>
+#include <string>
+#include <unordered_map>
+
+#include "chrome/browser/chromeos/printing/cups_print_job_manager.h"
+#include "chrome/browser/chromeos/printing/cups_print_job_notification.h"
+
+class Profile;
+
+namespace chromeos {
+
+class CupsPrintJob;
+
+class CupsPrintJobNotificationManager : public CupsPrintJobManager::Observer {
+ public:
+  using PrintJobNotificationMap =
+      std::unordered_map<CupsPrintJob*,

[skau, 2016/10/27 01:43:01]

Can you use the CupsPrintJob unique id instead of a pointer as the index?

[xdai1, 2016/10/28 23:59:54]

Why? If we use pointer, we don't need to always look up print jobs table to find
a specific print job for a given id. 

[skau, 2016/10/31 15:39:05]

My main concern is that it requires any object that's going to interact with the
manager to hold a raw pointer to a CupsPrintJob that it doesn't own and we know
will be deleted.  If it doesn't cause problems in practice it's fine.  It just
makes me a bit nervous.

+                         std::unique_ptr<CupsPrintJobNotification>>;
+
+  CupsPrintJobNotificationManager(Profile* profile,
+                                  CupsPrintJobManager* print_job_manager);
+  ~CupsPrintJobNotificationManager() override;
+
+  // CupsPrintJobManager::Observer overrides:
+  void OnPrintJobCreated(CupsPrintJob* job) override;
+  void OnPrintJobStarted(CupsPrintJob* job) override;
+  void OnPrintJobUpdated(CupsPrintJob* job) override;
+  void OnPrintJobSuspended(CupsPrintJob* job) override;
+  void OnPrintJobResumed(CupsPrintJob* job) override;
+  void OnPrintJobDone(CupsPrintJob* job) override;
+  void OnPrintJobError(CupsPrintJob* job) override;
+
+ private:
+  PrintJobNotificationMap notification_map_;
+  CupsPrintJobManager* print_job_manager_;
+  Profile* profile_;
+
+  DISALLOW_COPY_AND_ASSIGN(CupsPrintJobNotificationManager);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_PRINTING_CUPS_PRINT_JOB_NOTIFICATION_MANAGER_H_