Move .ico hashing to icon_util

ui/gfx/icon_util.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/gfx/icon_util.h"
 
 #include "base/file_util.h"
 #include "base/logging.h"
+#include "base/md5.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/win/resource_util.h"
 #include "base/win/scoped_gdi_object.h"
 #include "base/win/scoped_handle.h"
 #include "base/win/scoped_hdc.h"
 #include "skia/ext/image_operations.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "ui/gfx/gdi_util.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/image/image_family.h"
 #include "ui/gfx/size.h"
 
 namespace {
 
+const base::FilePath::CharType kIconChecksumFileExt[] =
+    FILE_PATH_LITERAL(".ico.md5");
+
 struct ScopedICONINFO : ICONINFO {
   ScopedICONINFO() {
     hbmColor = NULL;
     hbmMask = NULL;
   }
 
   ~ScopedICONINFO() {
     if (hbmColor)
       ::DeleteObject(hbmColor);
     if (hbmMask)
@@ skipping 667 matching lines @@
   // Once we compute the size for a singe AND mask scan line, we multiply that
   // number by the image height in order to get the total number of bytes for
   // the AND mask. Thus, for a 15X15 image, we need 15 * 4 which is 60 bytes
   // for the monochrome bitmap representing the AND mask.
   size_t and_line_length = (bitmap.width() + 7) >> 3;
   and_line_length = (and_line_length + 3) & ~3;
   size_t and_mask_size = and_line_length * bitmap.height();
   size_t masks_size = *xor_mask_size + and_mask_size;
   *bytes_in_resource = masks_size + sizeof(BITMAPINFOHEADER);
 }
+
+void IconUtil::GetImageCheckSum(const gfx::ImageFamily& image,
+                                base::MD5Digest* digest) {
+  DCHECK(digest);
+  base::MD5Context md5_context;
+  base::MD5Init(&md5_context);
+
+  for (gfx::ImageFamily::const_iterator it = image.begin(); it != image.end();
+       ++it) {
+    SkBitmap bitmap = it->AsBitmap();
+
+    SkAutoLockPixels image_lock(bitmap);
+    base::StringPiece image_data(
+        reinterpret_cast<const char*>(bitmap.getPixels()), bitmap.getSize());
+    base::MD5Update(&md5_context, image_data);
+  }
+
+  base::MD5Final(digest, &md5_context);
+}
+
+bool IconUtil::SaveIconWithCheckSum(const base::FilePath& icon_file,
+                                    const gfx::ImageFamily& image) {
+  if (!CreateIconFileFromImageFamily(image, icon_file))
+    return false;
+
+  base::MD5Digest digest;
+  GetImageCheckSum(image, &digest);
+
+  base::FilePath cheksum_file(icon_file.ReplaceExtension(kIconChecksumFileExt));

[Alexei Svitkine (slow), 2013/05/09 15:34:57]

Nit: cheksum -> checksum

+  return file_util::WriteFile(cheksum_file,
+                              reinterpret_cast<const char*>(&digest),
+                              sizeof(digest)) == sizeof(digest);
+}
+
+bool IconUtil::ShouldUpdateIcon(const base::FilePath& icon_file,
+                      const gfx::ImageFamily& image) {

[Alexei Svitkine (slow), 2013/05/09 15:34:57]

Nit: Align.

+  base::FilePath checksum_file(
+      icon_file.ReplaceExtension(kIconChecksumFileExt));
+
+  // Returns true if icon_file or checksum file is missing.
+  if (!file_util::PathExists(icon_file) ||
+      !file_util::PathExists(checksum_file))
+    return true;
+
+  base::MD5Digest persisted_image_checksum;
+  if (sizeof(persisted_image_checksum) != file_util::ReadFile(checksum_file,
+                      reinterpret_cast<char*>(&persisted_image_checksum),
+                      sizeof(persisted_image_checksum)))
+    return true;
+
+  base::MD5Digest downloaded_image_checksum;
+  GetImageCheckSum(image, &downloaded_image_checksum);
+
+  // Update icon if checksums are not equal.
+  return memcmp(&persisted_image_checksum, &downloaded_image_checksum,
+                sizeof(base::MD5Digest)) != 0;
+}