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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ios/chrome/browser/signin/browser_state_data_remover.mm

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') | « ios/chrome/browser/ios_chrome_main_parts.mm ('k') | ios/web/web_state/ui/crw_web_controller.mm » ('j') | remoting/host/curtain_mode_mac.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/signin/browser_state_data_remover.mm
diff --git a/ios/chrome/browser/signin/browser_state_data_remover.mm b/ios/chrome/browser/signin/browser_state_data_remover.mm
index 7a3ba5c81e575997b5204577a005b2f803830a10..f8dec627af5d2dfd56acf8a1e05d4a6445b12488 100644
--- a/ios/chrome/browser/signin/browser_state_data_remover.mm
+++ b/ios/chrome/browser/signin/browser_state_data_remover.mm
@@ -64,8 +64,8 @@ void BrowserStateDataRemover::NotifyWithDetails(
// Remove bookmarks once all browsing data was removed.
// Removal of browsing data waits for the bookmark model to be loaded, so
// there should be no issue calling the function here.
- CHECK(RemoveAllUserBookmarksIOS(browser_state_))
- << "Failed to remove all user bookmarks.";
+ // Failed to remove all user bookmarks.
+ CHECK(RemoveAllUserBookmarksIOS(browser_state_));
if (details.removal_mask != kRemoveAllDataMask) {
NOTREACHED() << "Unexpected partial remove browsing data request "
« chrome/browser/autofill/form_structure_browsertest.cc ('K') | « ios/chrome/browser/ios_chrome_main_parts.mm ('k') | ios/web/web_state/ui/crw_web_controller.mm » ('j') | remoting/host/curtain_mode_mac.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698