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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/cookies/cookie_monster.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/base/linked_hash_map.h ('k') | net/http2/hpack/decoder/hpack_entry_collector.cc » ('j') | remoting/host/curtain_mode_mac.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/cookies/cookie_monster.cc
diff --git a/net/cookies/cookie_monster.cc b/net/cookies/cookie_monster.cc
index b656d83e6c078a0594dfd3c0f76d87ca0ea78488..25a84d02f24b15c590a98942302f996b8614d481 100644
--- a/net/cookies/cookie_monster.cc
+++ b/net/cookies/cookie_monster.cc
@@ -1653,8 +1653,8 @@ bool CookieMonster::DeleteAnyEquivalentCookie(const std::string& key,
// overwrite each other, unless secure cookies require secure scheme is
// being enforced. In that case, cookies with different paths might exist
// and be considered equivalent.
- CHECK(!found_equivalent_cookie)
- << "Duplicate equivalent cookies found, cookie store is corrupted.";
+ // Duplicate equivalent cookies found, cookie store is corrupted.
+ CHECK(!found_equivalent_cookie);
if (skip_httponly && cc->IsHttpOnly()) {
skipped_httponly = true;
} else {
« chrome/browser/autofill/form_structure_browsertest.cc ('K') | « net/base/linked_hash_map.h ('k') | net/http2/hpack/decoder/hpack_entry_collector.cc » ('j') | remoting/host/curtain_mode_mac.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698