Skip flushes on icon writes for 10x better jumplist perf

chrome/browser/jumplist_win.cc

Index: chrome/browser/jumplist_win.cc
diff --git a/chrome/browser/jumplist_win.cc b/chrome/browser/jumplist_win.cc
index ba78502d17eafa5540ab6590e9efcc81ac8ddc2e..9774bde361b6e40d63037926717df001e23ec830 100644
--- a/chrome/browser/jumplist_win.cc
+++ b/chrome/browser/jumplist_win.cc
@@ -82,7 +82,8 @@ bool CreateIconFile(const SkBitmap& bitmap,
   // save it as the temporary file.
   gfx::ImageFamily image_family;
   image_family.Add(gfx::Image::CreateFrom1xBitmap(bitmap));
-  if (!IconUtil::CreateIconFileFromImageFamily(image_family, path))
+  if (!IconUtil::CreateIconFileFromImageFamily(image_family, path,
+                                               IconUtil::NORMAL_WRITE))

[Alexei Svitkine (slow), 2015/09/30 21:26:23]

Nit: {}'s

[brucedawson, 2015/09/30 22:13:59]

Style guide says that {}'s are optional on single-line statements, and line 79
omits them so I think they should be omitted for consistency.

     return false;
 
   // Add this icon file to the list and return its absolute path.