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

Side by Side Diff: printing/backend/print_backend.h

Issue 2463473002: Present the printer list from preferences for Chrome OS. (Closed)
Patch Set: tidy Created 4 years, 1 month 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
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef PRINTING_BACKEND_PRINT_BACKEND_H_ 5 #ifndef PRINTING_BACKEND_PRINT_BACKEND_H_
6 #define PRINTING_BACKEND_PRINT_BACKEND_H_ 6 #define PRINTING_BACKEND_PRINT_BACKEND_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 10 matching lines...) Expand all
21 // This is the interface for platform-specific code for a print backend 21 // This is the interface for platform-specific code for a print backend
22 namespace printing { 22 namespace printing {
23 23
24 // Note: There are raw values. The |printer_name| and |printer_description| 24 // Note: There are raw values. The |printer_name| and |printer_description|
25 // require further interpretation on Mac. See existing callers for examples. 25 // require further interpretation on Mac. See existing callers for examples.
26 struct PRINTING_EXPORT PrinterBasicInfo { 26 struct PRINTING_EXPORT PrinterBasicInfo {
27 PrinterBasicInfo(); 27 PrinterBasicInfo();
28 PrinterBasicInfo(const PrinterBasicInfo& other); 28 PrinterBasicInfo(const PrinterBasicInfo& other);
29 ~PrinterBasicInfo(); 29 ~PrinterBasicInfo();
30 30
31 std::string printer_id;
Lei Zhang 2016/10/28 21:55:51 Is this field going to be ChromeOS only?
skau 2016/11/02 22:00:03 I've added a guard for now. It's probably useful
Lei Zhang 2016/11/02 22:40:59 Ok, so in patch set 2 you had [printer_id, printer
skau 2016/11/02 23:38:16 Sorry about the thrashing. display_name is more c
31 std::string printer_name; 32 std::string printer_name;
32 std::string printer_description; 33 std::string printer_description;
33 int printer_status; 34 int printer_status;
34 int is_default; 35 int is_default;
35 std::map<std::string, std::string> options; 36 std::map<std::string, std::string> options;
36 }; 37 };
37 38
38 using PrinterList = std::vector<PrinterBasicInfo>; 39 using PrinterList = std::vector<PrinterBasicInfo>;
39 40
40 struct PRINTING_EXPORT PrinterSemanticCapsAndDefaults { 41 struct PRINTING_EXPORT PrinterSemanticCapsAndDefaults {
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 static void SetNativeCupsEnabled(bool enabled); 130 static void SetNativeCupsEnabled(bool enabled);
130 131
131 protected: 132 protected:
132 friend class base::RefCountedThreadSafe<PrintBackend>; 133 friend class base::RefCountedThreadSafe<PrintBackend>;
133 virtual ~PrintBackend(); 134 virtual ~PrintBackend();
134 }; 135 };
135 136
136 } // namespace printing 137 } // namespace printing
137 138
138 #endif // PRINTING_BACKEND_PRINT_BACKEND_H_ 139 #endif // PRINTING_BACKEND_PRINT_BACKEND_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698