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

Unified Diff: net/tools/tld_cleanup/tld_cleanup_util.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: net/tools/tld_cleanup/tld_cleanup_util.cc
diff --git a/net/tools/tld_cleanup/tld_cleanup_util.cc b/net/tools/tld_cleanup/tld_cleanup_util.cc
index 6ed431f11773208bd77813018cf0ac9cce4c49c3..f83a973822836bcf96d2d43b95de8a7ddd02a976 100644
--- a/net/tools/tld_cleanup/tld_cleanup_util.cc
+++ b/net/tools/tld_cleanup/tld_cleanup_util.cc
@@ -177,8 +177,8 @@ NormalizeResult NormalizeDataToRuleMap(const std::string data,
// wildcard for the same rule, or that the same domain is listed as both
// private and not private. If we did, we'd have to update our
// parsing code to handle this case.
- CHECK(rules->find(domain) == rules->end())
- << "Duplicate rule found for " << domain;
+ // Duplicate rule found for |domain|.
+ CHECK(rules->find(domain) == rules->end());
(*rules)[domain] = rule;
// Add true TLD for multi-level rules. We don't add them right now, in

Powered by Google App Engine
This is Rietveld 408576698