Delete JumpListIconsOld and update revised links in every update

chrome/browser/win/jumplist_file_util.h

Index: chrome/browser/win/jumplist_file_util.h
diff --git a/chrome/browser/win/jumplist_file_util.h b/chrome/browser/win/jumplist_file_util.h
index ec618162aa256fcb35c3043ea52cd6b481928141..9e19793443215e630e8f720ab712c0dff98eedd1 100644
--- a/chrome/browser/win/jumplist_file_util.h
+++ b/chrome/browser/win/jumplist_file_util.h
@@ -5,7 +5,7 @@
 #ifndef CHROME_BROWSER_WIN_JUMPLIST_FILE_UTIL_H_
 #define CHROME_BROWSER_WIN_JUMPLIST_FILE_UTIL_H_
 
-#include "base/files/file_path.h"
+#include "base/files/file_util.h"

[grt (UTC plus 2), 2017/03/27 08:03:39]

i believe file_path.h is still correct based on iwyu -- file_util.h should be
included by the .cc file

[chengx, 2017/03/27 18:32:41]

Done.

[grt (UTC plus 2), 2017/03/27 19:18:47]

ping

[chengx, 2017/03/27 20:26:08]

Ah, sorry... Now it is fixed.

 
 // Maximum number of icon files allowed to delete per jumplist update.
 const int kFileDeleteLimit = 100;
@@ -36,14 +36,15 @@ enum FolderDeleteResult {
   END
 };
 
-// An enumeration indicating if a directory is empty or not.
+// An enumeration indicating if a directory exists or if it is empty or not.
 // This is used for UMA. Do not delete entries, and keep in sync with
 // histograms.xml.
-enum DirectoryEmptyStatus {
+enum DirectoryStatus {
   EMPTY = 0,
   NON_EMPTY,
+  NON_EXIST,
   // Add new items before this one, always keep this one at the end.
-  EMPTY_STATUS_END
+  DIRECTORY_STATUS_END
 };
 
 // This method is similar to base::DeleteFileRecursive in