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

Unified Diff: chrome/browser/ui/webui/signin/user_manager_screen_handler.cc

Issue 2040473002: Move {Resolve|Reject}JavascriptCallback helpers in WebUIMessageHandler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Resolvig conflicts with ToT 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/signin/user_manager_screen_handler.cc
diff --git a/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc b/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
index c2850274668d202bb372f5d45ff9c805e8633675..9562da076662b2035c6d9b7f44dbd9c1fb0b67cf 100644
--- a/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
+++ b/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
@@ -502,9 +502,9 @@ void UserManagerScreenHandler::HandleAreAllProfilesLocked(
bool success = args->GetString(0, &webui_callback_id);
DCHECK(success);
- web_ui()->CallJavascriptFunctionUnsafe(
- "cr.webUIResponse", base::StringValue(webui_callback_id),
- base::FundamentalValue(true),
+ AllowJavascript();
+ ResolveJavascriptCallback(
+ base::StringValue(webui_callback_id),
base::FundamentalValue(profiles::AreAllProfilesLocked()));
}

Powered by Google App Engine
This is Rietveld 408576698