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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/win/jumplist_file_util.cc

Issue 2827563002: Merge jumplist delete task and update task (Closed)
Patch Set: Fix typo Created 3 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
« no previous file with comments | « chrome/browser/win/jumplist.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/win/jumplist_file_util.cc
diff --git a/chrome/browser/win/jumplist_file_util.cc b/chrome/browser/win/jumplist_file_util.cc
index 7c9870441d73388df3cc79f22f9977169ac4672e..9dd5b11e3041070d2fad91ac470742862438e25f 100644
--- a/chrome/browser/win/jumplist_file_util.cc
+++ b/chrome/browser/win/jumplist_file_util.cc
@@ -125,6 +125,9 @@ void DeleteDirectoryAndLogResults(const base::FilePath& path,
void DeleteDirectoryContentAndLogResults(const base::FilePath& path,
int max_file_deleted) {
+ // TODO(chengx): Remove the UMA histogram after fixing http://crbug.com/40407.
+ SCOPED_UMA_HISTOGRAM_TIMER("WinJumplist.DeleteDirectoryContentDuration");
+
DirectoryStatus dir_status = NON_EXIST;
// Delete the content in |path|. If |path| doesn't exist, create one.
« no previous file with comments | « chrome/browser/win/jumplist.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698