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

Side by Side Diff: chromeos/printing/printer_translator.cc

Issue 2161933003: Define printer configuration objects for Chrome OS. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Done Created 4 years, 4 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chromeos/printing/printer_translator.h"
6
7 #include <memory>
8 #include <string>
9 #include <utility>
10 #include <vector>
11
12 #include "base/memory/ptr_util.h"
13 #include "base/values.h"
14 #include "chromeos/printing/printer_configuration.h"
15
16 using base::DictionaryValue;
17
18 namespace chromeos {
19
20 namespace {
21
22 // ppd fields
23 const char kPPDid[] = "id";
24 const char kFileName[] = "file_name";
25 const char kVersionNumber[] = "version_number";
26 const char kFromQuirks[] = "from_quirks_server";
27
28 // printer fields
29 const char kDisplayName[] = "display_name";
30 const char kDescription[] = "description";
31 const char kManufacturer[] = "manufacturer";
32 const char kModel[] = "model";
33 const char kUri[] = "uri";
34 const char kPPD[] = "ppd";
35 const char kUUID[] = "uuid";
36
37 void MergePPDFile(const DictionaryValue* value, Printer::PPDFile* ppd) {
38 int id = -1;
39 value->GetInteger(kPPDid, &id);
40 ppd->id = id;
41
42 std::string file_name;
43 if (value->GetString(kFileName, &file_name))
44 ppd->file_name = file_name;
45
46 int version_number = 0;
47 value->GetInteger(kVersionNumber, &version_number);
Lei Zhang 2016/08/02 02:35:52 Check return value like you do above and below?
skau 2016/08/02 20:17:22 Done.
48 ppd->version_number = version_number;
49
50 bool from_quirks;
51 if (value->GetBoolean(kFromQuirks, &from_quirks))
52 ppd->from_quirks_server = from_quirks;
53 }
54
55 void PPDFileToDictionary(const Printer::PPDFile& ppd,
Lei Zhang 2016/08/02 02:35:52 Same comment about returning a std::unique_ptr<bas
skau 2016/08/02 20:17:22 Done.
56 DictionaryValue* dictionary) {
57 dictionary->SetInteger(kPPDid, ppd.id);
58 dictionary->SetString(kFileName, ppd.file_name);
59 dictionary->SetInteger(kVersionNumber, ppd.version_number);
60 dictionary->SetBoolean(kFromQuirks, ppd.from_quirks_server);
61 }
62
63 } // namespace
64
65 namespace printing {
66
67 const char kPrinterId[] = "id";
68
69 bool MergePrinterPreference(const DictionaryValue& value, Printer* printer) {
70 std::string id;
71 if (!value.GetString(kPrinterId, &id)) {
72 LOG(WARNING) << "Record id required";
73 return false;
74 }
75
76 printer->set_id(id);
77
78 std::string display_name;
79 if (value.GetString(kDisplayName, &display_name))
80 printer->set_display_name(display_name);
81
82 std::string description;
83 if (value.GetString(kDescription, &description))
84 printer->set_description(description);
85
86 std::string manufacturer;
87 if (value.GetString(kManufacturer, &manufacturer))
88 printer->set_manufacturer(manufacturer);
89
90 std::string model;
91 if (value.GetString(kModel, &model))
92 printer->set_model(model);
93
94 std::string uri;
95 if (value.GetString(kUri, &uri))
96 printer->set_uri(uri);
97
98 std::string uuid;
99 if (value.GetString(kUUID, &uuid))
100 printer->set_uuid(uuid);
101
102 const DictionaryValue* ppd;
103 if (value.GetDictionary(kPPD, &ppd)) {
104 // TODO(skau): merge?
105 Printer::PPDFile ppdFile;
106 MergePPDFile(ppd, &ppdFile);
107 printer->set_ppd(ppdFile);
108 }
109
110 return true;
111 }
112
113 void PrinterToPref(const Printer& printer, DictionaryValue* dictionary) {
114 dictionary->SetString(kPrinterId, printer.id());
115 dictionary->SetString(kDisplayName, printer.display_name());
116 dictionary->SetString(kDescription, printer.description());
117 dictionary->SetString(kManufacturer, printer.manufacturer());
118 dictionary->SetString(kModel, printer.model());
119 dictionary->SetString(kUri, printer.uri());
120 dictionary->SetString(kUUID, printer.uuid());
121
122 std::unique_ptr<DictionaryValue> ppd = base::MakeUnique<DictionaryValue>();
123 PPDFileToDictionary(printer.ppd(), ppd.get());
124 // create a std::unique_ptr<Value> from ppd since
125 // std::unique_ptr<DictionaryValue> and std::unique_ptr<Value> are not
126 // compatible types.
127 dictionary->Set(kPPD, std::unique_ptr<base::Value>(std::move(ppd)));
Lei Zhang 2016/08/02 02:35:52 Does this work? dictionary->Set(kPPD, std::move(p
skau 2016/08/02 20:17:22 That does work. Neat.
128 }
129
130 } // namespace printing
131
132 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698