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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/url_request/url_request_context.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') | « net/tools/tld_cleanup/tld_cleanup_util.cc ('k') | net/websockets/websocket_basic_stream.cc » ('j') | remoting/host/curtain_mode_mac.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/url_request/url_request_context.cc
diff --git a/net/url_request/url_request_context.cc b/net/url_request/url_request_context.cc
index 42bcc899f06335addb72b43c43b1e6d9286143d4..21a3e2dda0233df838a2caa1eb44d7dc69566c8e 100644
--- a/net/url_request/url_request_context.cc
+++ b/net/url_request/url_request_context.cc
@@ -113,8 +113,8 @@ void URLRequestContext::AssertNoURLRequests() const {
base::debug::Alias(url_buf);
base::debug::Alias(&num_requests);
base::debug::Alias(&load_flags);
- CHECK(false) << "Leaked " << num_requests << " URLRequest(s). First URL: "
- << request->url().spec().c_str() << ".";
+ LOG(FATAL) << "Leaked " << num_requests
+ << " URLRequest(s). First URL: " << request->url().spec() << ".";
}
}
« chrome/browser/autofill/form_structure_browsertest.cc ('K') | « net/tools/tld_cleanup/tld_cleanup_util.cc ('k') | net/websockets/websocket_basic_stream.cc » ('j') | remoting/host/curtain_mode_mac.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698