0

Add PrinterType enum to print.mojom

crrev.com/c/2944339 adds a GetPrinterTypeDenyList() method to the
LocalPrinter interface which returns a list of PrinterType enums.

Bug: 1192973, 1222453
Change-Id: Ic984507f9e3aaa776720f4cd55b9c0b099c3dfba
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2977411
Auto-Submit: Pranav Batra <batrapranav@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Pranav Batra <batrapranav@chromium.org>
Cr-Commit-Position: refs/heads/master@{#894818}
This commit is contained in:
Pranav Batra
2021-06-22 19:23:18 +00:00
committed by Chromium LUCI CQ
parent 8f4d51b562
commit 20a427d7b7
3 changed files with 22 additions and 4 deletions

@ -56,7 +56,12 @@ component("printing_base") {
cflags = []
defines = [ "IS_PRINTING_BASE_IMPL" ]
public_deps = [ "//build:chromeos_buildflags" ]
public_deps = [
"//build:chromeos_buildflags",
# TODO(crbug.com/1222453): Remove this dependency once migration is complete.
"//printing/mojom",
]
deps = [
"//base",
"//third_party/icu",

@ -135,3 +135,14 @@ enum ResultCode {
// Insufficient permissions to perform the operation.
kAccessDenied,
};
// Matches print_preview.PrinterType in
// chrome/browser/resources/print_preview/data/destination_match.js
[Stable, Extensible]
enum PrinterType {
kPrivet,
kExtension,
kPdf,
kLocal,
kCloud
};

@ -9,6 +9,9 @@
#include "base/component_export.h"
// TODO(crbug.com/1222453): Remove once migration is complete.
#include "printing/mojom/print.mojom.h"
namespace printing {
COMPONENT_EXPORT(PRINTING_BASE) extern const char kIsFirstRequest[];
@ -186,9 +189,8 @@ enum ScalingType {
SCALING_TYPE_LAST = CUSTOM
};
// Must match print_preview.PrinterType in
// chrome/browser/resources/print_preview/data/destination_match.js
enum class PrinterType { kPrivet, kExtension, kPdf, kLocal, kCloud };
// TODO(crbug.com/1222453): Remove once migration is complete.
using PrinterType = printing::mojom::PrinterType;
} // namespace printing