Change 4 funcs to JumpList member funcs, retire wstring for jumplist*

chrome/browser/win/jumplist.h

Index: chrome/browser/win/jumplist.h
diff --git a/chrome/browser/win/jumplist.h b/chrome/browser/win/jumplist.h
index d5af79786010255a86d9d282b564b8a2e25638ff..525f04ca8da1c3fbd68966766df0e7b6d62a7c29 100644
--- a/chrome/browser/win/jumplist.h
+++ b/chrome/browser/win/jumplist.h
@@ -177,6 +177,40 @@ class JumpList : public sessions::TabRestoreServiceObserver,
   // after requests storms have subsided.
   void DeferredTabRestoreServiceChanged();
 
+  // Creates at most |max_items| icon files 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);
+
+  // 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);
+
+  // Updates the jumplist, once all the data has been fetched. This method calls
+  // UpdateJumpList() to do most of the work.
+  void RunUpdateJumpList(
+      IncognitoModePrefs::Availability incognito_availability,
+      const std::wstring& app_id,

[gab, 2017/05/09 14:54:28]

Use base::string16 instead of std::wstring (I'm surprised this didn't hit the
presubmit against wstring [1]?)

And #include "base/strings/string16.h"

[1]
https://cs.chromium.org/chromium/src/PRESUBMIT.py?rcl=f41443c03b03d8e2d83314a...

[chengx, 2017/05/09 18:04:14]

Done. It did hit the presubmit against wstring, but I thought it was okay for
Windows platform. I was not aware we have base::string16. My bad.

+      const base::FilePath& profile_dir,
+      base::RefCountedData<JumpListData>* ref_counted_data);
+
+  // Updates the application JumpList, which consists of 1) delete old icon
+  // files; 2) create new icon files; 3) notify the OS. This method is called
+  // from RunUpdateJumpList().
+  // Note that any timeout error along the way results in the old jumplist being
+  // left as-is, while any non-timeout error results in the old jumplist being
+  // left as-is, but without icon files.
+  bool UpdateJumpList(const wchar_t* app_id,
+                      const base::FilePath& profile_dir,
+                      const ShellLinkItemList& most_visited_pages,
+                      const ShellLinkItemList& recently_closed_pages,
+                      bool most_visited_pages_have_updates,
+                      bool recently_closed_pages_have_updates,
+                      IncognitoModePrefs::Availability incognito_availability);
+
   // Tracks FaviconService tasks.
   base::CancelableTaskTracker cancelable_task_tracker_;