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

Unified Diff: chrome/browser/ui/webui/print_preview/printer_backend_proxy_chromeos.cc

Issue 2906633004: Modify PrinterSetupResult enum to constant style. (Closed)
Patch Set: git squash commit for enum-style. 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
Index: chrome/browser/ui/webui/print_preview/printer_backend_proxy_chromeos.cc
diff --git a/chrome/browser/ui/webui/print_preview/printer_backend_proxy_chromeos.cc b/chrome/browser/ui/webui/print_preview/printer_backend_proxy_chromeos.cc
index a26a8bbc8891c4cd1e3d8b6760c6c7b3800598de..d4372fbdc75791038f2382eb949dd0b6be7f75d7 100644
--- a/chrome/browser/ui/webui/print_preview/printer_backend_proxy_chromeos.cc
+++ b/chrome/browser/ui/webui/print_preview/printer_backend_proxy_chromeos.cc
@@ -125,7 +125,7 @@ class PrinterBackendProxyChromeos : public PrinterBackendProxy {
if (prefs_->IsConfigurationCurrent(*printer)) {
// Skip setup if the printer is already installed.
- HandlePrinterSetup(std::move(printer), cb, chromeos::SUCCESS);
+ HandlePrinterSetup(std::move(printer), cb, chromeos::kSuccess);
return;
}
@@ -143,7 +143,7 @@ class PrinterBackendProxyChromeos : public PrinterBackendProxy {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
switch (result) {
- case chromeos::PrinterSetupResult::SUCCESS:
+ case chromeos::PrinterSetupResult::kSuccess:
VLOG(1) << "Printer setup successful for " << printer->id()
<< " fetching properties";
prefs_->PrinterInstalled(*printer);
@@ -151,21 +151,21 @@ class PrinterBackendProxyChromeos : public PrinterBackendProxy {
// fetch settings on the blocking pool and invoke callback.
FetchCapabilities(std::move(printer), cb);
return;
- case chromeos::PrinterSetupResult::PPD_NOT_FOUND:
+ case chromeos::PrinterSetupResult::kPpdNotFound:
LOG(WARNING) << "Could not find PPD. Check printer configuration.";
// Prompt user to update configuration.
// TODO(skau): Fill me in
break;
- case chromeos::PrinterSetupResult::PPD_UNRETRIEVABLE:
+ case chromeos::PrinterSetupResult::kPpdUnretrievable:
LOG(WARNING) << "Could not download PPD. Check Internet connection.";
// Could not download PPD. Connect to Internet.
// TODO(skau): Fill me in
break;
- case chromeos::PrinterSetupResult::PRINTER_UNREACHABLE:
- case chromeos::PrinterSetupResult::DBUS_ERROR:
- case chromeos::PrinterSetupResult::PPD_TOO_LARGE:
- case chromeos::PrinterSetupResult::INVALID_PPD:
- case chromeos::PrinterSetupResult::FATAL_ERROR:
+ case chromeos::PrinterSetupResult::kPrinterUnreachable:
+ case chromeos::PrinterSetupResult::kDbusError:
+ case chromeos::PrinterSetupResult::kPpdTooLarge:
+ case chromeos::PrinterSetupResult::kInvalidPpd:
+ case chromeos::PrinterSetupResult::kFatalError:
LOG(ERROR) << "Unexpected error in printer setup." << result;
break;
}

Powered by Google App Engine
This is Rietveld 408576698