[CUPS] Implement the Print Job notification.

chrome/browser/chromeos/printing/cups_print_job.cc

+// 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.
+
+#include "chrome/browser/chromeos/printing/cups_print_job.h"
+
+#include "base/strings/string_number_conversions.h"
+
+namespace chromeos {
+
+CUPSPrintJob::CUPSPrintJob(const std::string& printer_id,
+                           int job_id,
+                           const std::string& document_title,
+                           int total_page_number)
+    : printer_(Printer(printer_id)),
+      job_id_(job_id),
+      document_title_(document_title),
+      total_page_number_(total_page_number) {}

[stevenjb, 2016/10/26 19:59:57]

Can we eliminate this and just make callers do the Printer(printer_id) call? In
general we prefer to avoid multiple constructors.

[xdai1, 2016/10/26 22:55:18]

Done.

+
+CUPSPrintJob::CUPSPrintJob(const Printer& printer,
+                           int job_id,
+                           const std::string& document_title,
+                           int total_page_number)
+    : printer_(printer),
+      job_id_(job_id),
+      document_title_(document_title),
+      total_page_number_(total_page_number) {}
+
+CUPSPrintJob::~CUPSPrintJob() {}
+
+std::string CUPSPrintJob::GetId() const {
+  return printer_.id() + base::IntToString(job_id_);
+}
+
+Printer* CUPSPrintJob::GetPrinter() {
+  return &printer_;
+}
+
+}  // namespace chromeos