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

Unified Diff: chrome/browser/ui/webui/chromeos/choose_mobile_network_ui.cc

Issue 1995113002: Rename WebUI::CallJavascriptFunction to WebUI::CallJavascriptFunctionUnsafe (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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/chromeos/choose_mobile_network_ui.cc
diff --git a/chrome/browser/ui/webui/chromeos/choose_mobile_network_ui.cc b/chrome/browser/ui/webui/chromeos/choose_mobile_network_ui.cc
index 6b6d154dbc5d10b31ebed5416f55e095a0d00719..8ada35fe90341849a230701559228a704728a93d 100644
--- a/chrome/browser/ui/webui/chromeos/choose_mobile_network_ui.cc
+++ b/chrome/browser/ui/webui/chromeos/choose_mobile_network_ui.cc
@@ -175,7 +175,7 @@ void ChooseMobileNetworkHandler::DeviceListChanged() {
NET_LOG_EVENT("ChooseMobileNetwork", "Device is scanning for networks.");
scanning_ = true;
if (is_page_ready_)
- web_ui()->CallJavascriptFunction(kJsApiShowScanning);
+ web_ui()->CallJavascriptFunctionUnsafe(kJsApiShowScanning);
return;
}
scanning_ = false;
@@ -203,7 +203,7 @@ void ChooseMobileNetworkHandler::DeviceListChanged() {
}
}
if (is_page_ready_) {
- web_ui()->CallJavascriptFunction(kJsApiShowNetworks, networks_list_);
+ web_ui()->CallJavascriptFunctionUnsafe(kJsApiShowNetworks, networks_list_);
networks_list_.Clear();
has_pending_results_ = false;
} else {
@@ -252,11 +252,11 @@ void ChooseMobileNetworkHandler::HandlePageReady(const base::ListValue* args) {
}
if (has_pending_results_) {
- web_ui()->CallJavascriptFunction(kJsApiShowNetworks, networks_list_);
+ web_ui()->CallJavascriptFunctionUnsafe(kJsApiShowNetworks, networks_list_);
networks_list_.Clear();
has_pending_results_ = false;
} else if (scanning_) {
- web_ui()->CallJavascriptFunction(kJsApiShowScanning);
+ web_ui()->CallJavascriptFunctionUnsafe(kJsApiShowScanning);
}
is_page_ready_ = true;
}
« no previous file with comments | « chrome/browser/ui/webui/certificate_viewer_webui.cc ('k') | chrome/browser/ui/webui/chromeos/cryptohome_web_ui_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698