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

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

Issue 2886083002: MD Settings: Use FireWebUIListener() helper wherever possible. (Closed)
Patch Set: Fixed 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
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 fe90000b6233f1dd6891adbd1a16c4e0051f5495..28a1781f5cad216bd1123cd274008f7d44ca2c0e 100644
--- a/chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.cc
+++ b/chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.cc
@@ -289,9 +289,7 @@ void CupsPrintersHandler::OnAddedPrinter(
}
void CupsPrintersHandler::OnAddPrinterError() {
- CallJavascriptFunction("cr.webUIListenerCallback",
- base::Value("on-add-cups-printer"), base::Value(false),
- base::Value(""));
+ FireWebUIListener("on-add-cups-printer", base::Value(false), base::Value(""));
}
void CupsPrintersHandler::HandleGetCupsPrinterManufacturers(
@@ -409,13 +407,11 @@ void CupsPrintersHandler::OnPrintersFound(
printers_list->Append(GetPrinterInfo(printer));
}
- CallJavascriptFunction("cr.webUIListenerCallback",
- base::Value("on-printer-discovered"), *printers_list);
+ FireWebUIListener("on-printer-discovered", *printers_list);
}
void CupsPrintersHandler::OnDiscoveryInitialScanDone() {
- CallJavascriptFunction("cr.webUIListenerCallback",
- base::Value("on-printer-discovery-done"));
+ FireWebUIListener("on-printer-discovery-done");
}
} // namespace settings

Powered by Google App Engine
This is Rietveld 408576698