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

Unified Diff: chrome/browser/win/jumplist.h

Issue 2898083003: Log the number of new icons created per jumplist update (Closed)
Patch Set: Fix nits Created 3 years, 7 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 | « no previous file | chrome/browser/win/jumplist.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/win/jumplist.h
diff --git a/chrome/browser/win/jumplist.h b/chrome/browser/win/jumplist.h
index 77a19e3025bfda8abd2da1ba794e5191c5a595d6..a0fb0bf7ff55dad1bb963c26a4d114103842241a 100644
--- a/chrome/browser/win/jumplist.h
+++ b/chrome/browser/win/jumplist.h
@@ -189,17 +189,19 @@ class JumpList : public sessions::TabRestoreServiceObserver,
// Creates at most |max_items| icon files of |category| in |icon_dir| for the
// asynchrounously loaded icons stored in |item_list|.
- void CreateIconFiles(const base::FilePath& icon_dir,
- const ShellLinkItemList& item_list,
- size_t max_items,
- JumpListCategory category);
+ // Returns the number of new icon files created.
+ int CreateIconFiles(const base::FilePath& icon_dir,
+ const ShellLinkItemList& item_list,
+ size_t max_items,
+ JumpListCategory category);
// Updates icon files in |icon_dir|, which includes deleting old icons and
// creating at most |slot_limit| new icons for |page_list|.
- void UpdateIconFiles(const base::FilePath& icon_dir,
- const ShellLinkItemList& page_list,
- size_t slot_limit,
- JumpListCategory category);
+ // Returns the number of new icon files created.
+ int UpdateIconFiles(const base::FilePath& icon_dir,
+ const ShellLinkItemList& page_list,
+ size_t slot_limit,
+ JumpListCategory category);
// Updates the jumplist, once all the data has been fetched. This method calls
// UpdateJumpList() to do most of the work.
« no previous file with comments | « no previous file | chrome/browser/win/jumplist.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698