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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: extensions/browser/api/sockets_tcp/sockets_tcp_api.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
« chrome/browser/autofill/form_structure_browsertest.cc ('K') | « extensions/browser/api/alarms/alarm_manager.cc ('k') | extensions/browser/api/sockets_udp/sockets_udp_api.cc » ('j') | remoting/host/curtain_mode_mac.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/api/sockets_tcp/sockets_tcp_api.cc
diff --git a/extensions/browser/api/sockets_tcp/sockets_tcp_api.cc b/extensions/browser/api/sockets_tcp/sockets_tcp_api.cc
index 1d956797a2479a515b65e0a71d4f0f6207aaf3cc..e6342fd1c067f4f2a2914267c22566cea89ef44a 100644
--- a/extensions/browser/api/sockets_tcp/sockets_tcp_api.cc
+++ b/extensions/browser/api/sockets_tcp/sockets_tcp_api.cc
@@ -370,7 +370,8 @@ void SocketsTcpSendFunction::OnCompleted(int net_result) {
}
void SocketsTcpSendFunction::SetSendResult(int net_result, int bytes_sent) {
- CHECK(net_result <= net::OK) << "Network status code must be <= net::OK";
+ // Network status code must be <= net::OK
+ CHECK(net_result <= net::OK);
sockets_tcp::SendInfo send_info;
send_info.result_code = net_result;
« chrome/browser/autofill/form_structure_browsertest.cc ('K') | « extensions/browser/api/alarms/alarm_manager.cc ('k') | extensions/browser/api/sockets_udp/sockets_udp_api.cc » ('j') | remoting/host/curtain_mode_mac.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698