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

Unified Diff: chrome/browser/ui/browser.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/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index 434636764411e2cd30a0cab292636dc101fb509d..ce9a15bb1e985fb73c21e7c746c9cc2aef77e724 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -368,8 +368,8 @@ Browser::Browser(const CreateParams& params)
// that is disallowed by policy. The crash prevents the disabled window type
// from opening at all, but the path that triggered it should be fixed.
CHECK(IncognitoModePrefs::CanOpenBrowser(profile_));
- CHECK(!profile_->IsGuestSession() || profile_->IsOffTheRecord())
- << "Only off the record browser may be opened in guest mode";
+ // Only off the record browser may be opened in guest mode
+ CHECK(!profile_->IsGuestSession() || profile_->IsOffTheRecord());
DCHECK(!profile_->IsSystemProfile())
<< "The system profile should never have a real browser.";
// TODO(mlerman): After this hits stable channel, see if there are counts

Powered by Google App Engine
This is Rietveld 408576698