Added check for icons that change.

chrome/browser/icon_loader_win.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 "chrome/browser/icon_loader.h"
 
 #include <windows.h>
 #include <shellapi.h>
 
 #include "base/bind.h"
 #include "base/message_loop.h"
 #include "base/threading/thread.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "ui/gfx/icon_util.h"
 #include "ui/gfx/image/image_skia.h"
 #include "ui/gfx/size.h"
 
 // static
 IconGroupID IconLoader::ReadGroupIDFromFilepath(
     const base::FilePath& filepath) {
+  if (!IsIconMutableFromFilepath(filepath))
+    return filepath.Extension();
+  return filepath.value();
+}
+
+bool IconLoader::IsIconMutableFromFilepath(const base::FilePath& filepath) {
   base::FilePath::StringType extension = filepath.Extension();
-  if (extension != L".exe" && extension != L".dll" && extension != L".ico")
-    return extension;
-  else
-    return filepath.value();
+  if (extension == L".exe" || extension == L".dll" || extension == L".ico")

[Robert Sesek, 2013/04/09 20:13:58]

Why not just return the result of this boolean expression?

[shatch, 2013/04/09 20:43:25]

Done.

+    return true;
+  return false;
 }
 
 void IconLoader::ReadIcon() {
   int size = 0;
   switch (icon_size_) {
     case IconLoader::SMALL:
       size = SHGFI_SMALLICON;
       break;
     case IconLoader::NORMAL:
       size = 0;
@@ skipping 18 matching lines @@
       image_skia.MakeThreadSafe();
       image_.reset(new gfx::Image(image_skia));
       DestroyIcon(file_info.hIcon);
     }
   }
 
   // Always notify the delegate, regardless of success.
   target_message_loop_->PostTask(FROM_HERE,
       base::Bind(&IconLoader::NotifyDelegate, this));
 }