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

Unified Diff: chrome/browser/ui/webui/print_preview/print_preview_handler.h

Issue 1979903002: Print Preview: Get printer info with capabilities. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@cups_14
Patch Set: nit Created 4 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/print_preview_handler.h
diff --git a/chrome/browser/ui/webui/print_preview/print_preview_handler.h b/chrome/browser/ui/webui/print_preview/print_preview_handler.h
index 878cb1ea5b5d968fcc909a3f86d475882ea6408e..60d4278b7c0ecf7b561f5e3cc36c2ebe35d7a1c7 100644
--- a/chrome/browser/ui/webui/print_preview/print_preview_handler.h
+++ b/chrome/browser/ui/webui/print_preview/print_preview_handler.h
@@ -209,12 +209,11 @@ class PrintPreviewHandler
const std::string& access_token);
// Sends the printer capabilities to the Web UI. |settings_info| contains
- // printer capabilities information.
- void SendPrinterCapabilities(const base::DictionaryValue* settings_info);
-
- // Sends error notification to the Web UI when unable to return the printer
- // capabilities.
- void SendFailedToGetPrinterCapabilities(const std::string& printer_name);
+ // printer capabilities information. If |settings_info| is empty, sends
+ // error notification to the Web UI instead.
+ void SendPrinterCapabilities(
+ const std::string& printer_name,
+ std::unique_ptr<base::DictionaryValue> settings_info);
// Send the list of printers to the Web UI.
void SetupPrinterList(const base::ListValue* printers);

Powered by Google App Engine
This is Rietveld 408576698