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

Unified Diff: chrome/browser/profile_resetter/profile_resetter.cc

Issue 1107863002: [chrome/browser/profile_resetter] favor DCHECK_CURRENTLY_ON for better logs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 months 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/profile_resetter/profile_resetter.cc
diff --git a/chrome/browser/profile_resetter/profile_resetter.cc b/chrome/browser/profile_resetter/profile_resetter.cc
index 9a5ba3fd96bc9c6223b742a20a400f009f250abc..ab2a026428c49c1c16447b1839b065c98cfb520c 100644
--- a/chrome/browser/profile_resetter/profile_resetter.cc
+++ b/chrome/browser/profile_resetter/profile_resetter.cc
@@ -37,7 +37,7 @@
namespace {
void ResetShortcutsOnFileThread() {
- DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
+ DCHECK_CURRENTLY_ON(content::BrowserThread::FILE);
// Get full path of chrome.
base::FilePath chrome_exe;
if (!PathService::Get(base::FILE_EXE, &chrome_exe))
@@ -127,7 +127,7 @@ void ProfileResetter::Reset(
}
bool ProfileResetter::IsActive() const {
- DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
+ DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
return pending_reset_flags_ != 0;
}
@@ -326,7 +326,7 @@ void ProfileResetter::OnBrowsingDataRemoverDone() {
std::vector<ShortcutCommand> GetChromeLaunchShortcuts(
const scoped_refptr<SharedCancellationFlag>& cancel) {
- DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
+ DCHECK_CURRENTLY_ON(content::BrowserThread::FILE);
#if defined(OS_WIN)
// Get full path of chrome.
base::FilePath chrome_exe;

Powered by Google App Engine
This is Rietveld 408576698