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

Unified Diff: chrome/browser/ui/webui/options/reset_profile_settings_handler.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/options/reset_profile_settings_handler.cc
diff --git a/chrome/browser/ui/webui/options/reset_profile_settings_handler.cc b/chrome/browser/ui/webui/options/reset_profile_settings_handler.cc
index baa498b1c4408a11e2282083b0c85d98663f4fce..d26b0893a509b71f7b8bb72772d6912a4d106d31 100644
--- a/chrome/browser/ui/webui/options/reset_profile_settings_handler.cc
+++ b/chrome/browser/ui/webui/options/reset_profile_settings_handler.cc
@@ -48,7 +48,7 @@ void ResetProfileSettingsHandler::InitializeHandler() {
}
void ResetProfileSettingsHandler::InitializePage() {
- web_ui()->CallJavascriptFunction(
+ web_ui()->CallJavascriptFunctionUnsafe(
"ResetProfileSettingsOverlay.setResettingState",
base::FundamentalValue(resetter_->IsActive()));
}
@@ -144,7 +144,8 @@ void ResetProfileSettingsHandler::HandleResetProfileSettings(
void ResetProfileSettingsHandler::OnResetProfileSettingsDone(
bool send_feedback) {
- web_ui()->CallJavascriptFunction("ResetProfileSettingsOverlay.doneResetting");
+ web_ui()->CallJavascriptFunctionUnsafe(
+ "ResetProfileSettingsOverlay.doneResetting");
if (send_feedback && setting_snapshot_) {
Profile* profile = Profile::FromWebUI(web_ui());
ResettableSettingsSnapshot current_snapshot(profile);
@@ -223,9 +224,8 @@ void ResetProfileSettingsHandler::UpdateFeedbackUI() {
Profile::FromWebUI(web_ui()), *setting_snapshot_);
base::DictionaryValue feedback_info;
feedback_info.Set("feedbackInfo", list.release());
- web_ui()->CallJavascriptFunction(
- "ResetProfileSettingsOverlay.setFeedbackInfo",
- feedback_info);
+ web_ui()->CallJavascriptFunctionUnsafe(
+ "ResetProfileSettingsOverlay.setFeedbackInfo", feedback_info);
}
} // namespace options

Powered by Google App Engine
This is Rietveld 408576698