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

Unified Diff: chrome/browser/profiles/profile_manager.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/profiles/profile_manager.cc
diff --git a/chrome/browser/profiles/profile_manager.cc b/chrome/browser/profiles/profile_manager.cc
index c05003344d63274343f5b7c5249adf0e37c064ed..8bba88ff260635e3628336ce00f18b04f92d7ce4 100644
--- a/chrome/browser/profiles/profile_manager.cc
+++ b/chrome/browser/profiles/profile_manager.cc
@@ -445,7 +445,7 @@ Profile* ProfileManager::GetActiveUserProfile() {
// However, most callers of this function immediately dereference the result
// which would lead to crashes in a variety of call sites. Assert here to
// figure out how common this is. http://crbug.com/383019
- CHECK(profile) << profile_manager->user_data_dir().AsUTF8Unsafe();
+ CHECK(profile);
return profile;
}
@@ -964,8 +964,8 @@ void ProfileManager::Observe(
// Confirm that we hadn't loaded the new profile previously.
base::FilePath default_profile_dir = user_data_dir_.Append(
GetInitialProfileDir());
- CHECK(!GetProfileByPathInternal(default_profile_dir))
- << "The default profile was loaded before we mounted the cryptohome.";
+ // The default profile was loaded before we mounted the cryptohome.
+ CHECK(!GetProfileByPathInternal(default_profile_dir));
}
return;
}

Powered by Google App Engine
This is Rietveld 408576698