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

Unified Diff: chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.cc

Issue 2906633004: Modify PrinterSetupResult enum to constant style. (Closed)
Patch Set: typo Created 3 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/webui/print_preview/printer_backend_proxy_chromeos.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.cc
diff --git a/chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.cc b/chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.cc
index 28a1781f5cad216bd1123cd274008f7d44ca2c0e..0a07cfacdb7c40c997af2f8f262a9bfcc2e7ef08 100644
--- a/chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.cc
+++ b/chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.cc
@@ -256,35 +256,38 @@ void CupsPrintersHandler::OnAddedPrinter(
chromeos::PrinterSetupResult result_code) {
std::string printer_name = printer->display_name();
switch (result_code) {
- case chromeos::PrinterSetupResult::SUCCESS: {
+ case chromeos::PrinterSetupResult::kSuccess: {
auto* manager = PrintersManagerFactory::GetForBrowserContext(profile_);
manager->PrinterInstalled(*printer);
manager->RegisterPrinter(std::move(printer));
break;
}
- case chromeos::PrinterSetupResult::PPD_NOT_FOUND:
+ case chromeos::PrinterSetupResult::kPpdNotFound:
LOG(WARNING) << "Could not locate requested PPD";
break;
- case chromeos::PrinterSetupResult::PPD_TOO_LARGE:
+ case chromeos::PrinterSetupResult::kPpdTooLarge:
LOG(WARNING) << "PPD is too large";
break;
- case chromeos::PrinterSetupResult::PPD_UNRETRIEVABLE:
+ case chromeos::PrinterSetupResult::kPpdUnretrievable:
LOG(WARNING) << "Could not retrieve PPD from server";
break;
- case chromeos::PrinterSetupResult::INVALID_PPD:
+ case chromeos::PrinterSetupResult::kInvalidPpd:
LOG(WARNING) << "Provided PPD is invalid.";
break;
- case chromeos::PrinterSetupResult::PRINTER_UNREACHABLE:
+ case chromeos::PrinterSetupResult::kPrinterUnreachable:
LOG(WARNING) << "Could not contact printer for configuration";
break;
- case chromeos::PrinterSetupResult::DBUS_ERROR:
- case chromeos::PrinterSetupResult::FATAL_ERROR:
+ case chromeos::PrinterSetupResult::kDbusError:
+ case chromeos::PrinterSetupResult::kFatalError:
LOG(ERROR) << "Unrecoverable error. Reboot required.";
break;
+ case chromeos::PrinterSetupResult::kMaxValue:
+ NOTREACHED() << "This is not an expected value";
+ break;
}
CallJavascriptFunction(
"cr.webUIListenerCallback", base::Value("on-add-cups-printer"),
- base::Value(result_code == chromeos::PrinterSetupResult::SUCCESS),
+ base::Value(result_code == chromeos::PrinterSetupResult::kSuccess),
base::Value(printer_name));
}
« no previous file with comments | « chrome/browser/ui/webui/print_preview/printer_backend_proxy_chromeos.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698