Define printer configuration objects for Chrome OS.

chrome/browser/chromeos/printing/printer_translator.h

Index: chrome/browser/chromeos/printing/printer_translator.h
diff --git a/chrome/browser/chromeos/printing/printer_translator.h b/chrome/browser/chromeos/printing/printer_translator.h
new file mode 100644
index 0000000000000000000000000000000000000000..6699412ddd3e409652cc518d5f195e9b68b61e8d
--- /dev/null
+++ b/chrome/browser/chromeos/printing/printer_translator.h
@@ -0,0 +1,28 @@
+// 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_PRINTER_TRANSLATOR_H_
+#define CHROME_BROWSER_CHROMEOS_PRINTING_PRINTER_TRANSLATOR_H_
+
+#include "chrome/browser/chromeos/printing/printer_configuration.h"
+
+namespace base {
+class DictionaryValue;
+}
+
+namespace chromeos {
+
+extern const char kPrinterId[];
+
+// Merge the fields in |pref| into |printer|.  Returns false if a required field
+// is missing.
+bool MergePrinterPreference(const base::DictionaryValue& pref,
+                            Printer* printer);
+
+// Updates the dictionary |pref| with the attributes of |printer|.
+void PrinterToPref(const Printer& printer, base::DictionaryValue* pref);

[stevenjb, 2016/07/29 16:56:51]

These should be in their own namespace, e.g. 'printing'.

[skau, 2016/08/01 19:37:21]

Just this or the Printer class too?

[stevenjb, 2016/08/01 20:48:16]

Classes are a namespace unto themselves, so putting them inside a 'printing'
namespace is optional, but globals variables and functions, which could
conceivably have meaning in another context (e.g. a chromeos settings class),
should be more specifically namespaced.

+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_PRINTING_PRINTER_TRANSLATOR_H_