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

Unified Diff: content/public/browser/web_ui_message_handler.h

Issue 2561963002: base: Remove the string logging from CHECK(). (Closed)
Patch Set: checkstring: rebase Created 4 years 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: content/public/browser/web_ui_message_handler.h
diff --git a/content/public/browser/web_ui_message_handler.h b/content/public/browser/web_ui_message_handler.h
index c8d3b09fcdd5fbc4bf0aa49dac6a753b336745b1..c97d8baf17fe84b12e108620a0e71d435834c280 100644
--- a/content/public/browser/web_ui_message_handler.h
+++ b/content/public/browser/web_ui_message_handler.h
@@ -100,8 +100,8 @@ class CONTENT_EXPORT WebUIMessageHandler {
template <typename... Values>
void CallJavascriptFunction(const std::string& function_name,
const Values&... values) {
- CHECK(IsJavascriptAllowed()) << "Cannot CallJavascriptFunction before "
- "explicitly allowing JavaScript.";
+ // Cannot CallJavascriptFunction before explicitly allowing JavaScript.
+ CHECK(IsJavascriptAllowed());
// The CHECK above makes this call safe.
web_ui()->CallJavascriptFunctionUnsafe(function_name, values...);

Powered by Google App Engine
This is Rietveld 408576698