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

Unified Diff: components/nacl/loader/nacl_helper_win_64.cc

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: components/nacl/loader/nacl_helper_win_64.cc
diff --git a/components/nacl/loader/nacl_helper_win_64.cc b/components/nacl/loader/nacl_helper_win_64.cc
index a17fe888a0a745637411c4949c973adff8e64c24..89444667906ce4a91a445d1f0e6d48681214af0e 100644
--- a/components/nacl/loader/nacl_helper_win_64.cc
+++ b/components/nacl/loader/nacl_helper_win_64.cc
@@ -83,8 +83,8 @@ int NaClWin64Main() {
// Initialize the sandbox for this process.
bool sandbox_initialized_ok = content::InitializeSandbox(&sandbox_info);
// Die if the sandbox can't be enabled.
- CHECK(sandbox_initialized_ok) << "Error initializing sandbox for "
- << process_type;
+ // Error initializing sandbox for |process_type|
+ CHECK(sandbox_initialized_ok);
content::MainFunctionParams main_params(command_line);
main_params.sandbox_info = &sandbox_info;
@@ -94,7 +94,8 @@ int NaClWin64Main() {
if (process_type == switches::kNaClBrokerProcess)
return NaClBrokerMain(main_params);
- CHECK(false) << "Unknown NaCl 64 process.";
+ // Unknown NaCl 64 process.
+ CHECK(false);
return -1;
}

Powered by Google App Engine
This is Rietveld 408576698