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

Unified Diff: chrome/browser/safe_browsing/certificate_reporting_service.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: chrome/browser/safe_browsing/certificate_reporting_service.cc
diff --git a/chrome/browser/safe_browsing/certificate_reporting_service.cc b/chrome/browser/safe_browsing/certificate_reporting_service.cc
index 52a20c7501cdc040239c5295ddf3d94269295877..0bd5b4f8417fc6bfae3469afc5c6cf02d06a1f5a 100644
--- a/chrome/browser/safe_browsing/certificate_reporting_service.cc
+++ b/chrome/browser/safe_browsing/certificate_reporting_service.cc
@@ -20,8 +20,8 @@ bool ReportCompareFunc(const CertificateReportingService::Report& item1,
CertificateReportingService::BoundedReportList::BoundedReportList(
size_t max_size)
: max_size_(max_size) {
- CHECK(max_size <= 20)
- << "Current implementation is not efficient for a large list.";
+ // Current implementation is not efficient for a large list.
+ CHECK(max_size <= 20);
DCHECK(thread_checker_.CalledOnValidThread());
}

Powered by Google App Engine
This is Rietveld 408576698