OLD | NEW |
---|---|
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "chrome/browser/icon_loader.h" | 5 #include "chrome/browser/icon_loader.h" |
6 | 6 |
7 #include <windows.h> | 7 #include <windows.h> |
8 #include <shellapi.h> | 8 #include <shellapi.h> |
9 | 9 |
10 #include "base/bind.h" | 10 #include "base/bind.h" |
11 #include "base/message_loop.h" | 11 #include "base/message_loop.h" |
12 #include "base/threading/thread.h" | 12 #include "base/threading/thread.h" |
13 #include "third_party/skia/include/core/SkBitmap.h" | 13 #include "third_party/skia/include/core/SkBitmap.h" |
14 #include "ui/gfx/icon_util.h" | 14 #include "ui/gfx/icon_util.h" |
15 #include "ui/gfx/image/image_skia.h" | 15 #include "ui/gfx/image/image_skia.h" |
16 #include "ui/gfx/size.h" | 16 #include "ui/gfx/size.h" |
17 | 17 |
18 // static | 18 // static |
19 IconGroupID IconLoader::ReadGroupIDFromFilepath( | 19 IconGroupID IconLoader::ReadGroupIDFromFilepath( |
20 const base::FilePath& filepath) { | 20 const base::FilePath& filepath) { |
21 if (!IsIconMutableFromFilepath(filepath)) | |
22 return filepath.Extension(); | |
23 return filepath.value(); | |
24 } | |
25 | |
26 bool IconLoader::IsIconMutableFromFilepath(const base::FilePath& filepath) { | |
21 base::FilePath::StringType extension = filepath.Extension(); | 27 base::FilePath::StringType extension = filepath.Extension(); |
22 if (extension != L".exe" && extension != L".dll" && extension != L".ico") | 28 return (extension == L".exe" || |
Robert Sesek
2013/04/09 20:44:10
nit: no () around return expressions
| |
23 return extension; | 29 extension == L".dll" || extension == L".ico"); |
24 else | |
25 return filepath.value(); | |
26 } | 30 } |
27 | 31 |
28 void IconLoader::ReadIcon() { | 32 void IconLoader::ReadIcon() { |
29 int size = 0; | 33 int size = 0; |
30 switch (icon_size_) { | 34 switch (icon_size_) { |
31 case IconLoader::SMALL: | 35 case IconLoader::SMALL: |
32 size = SHGFI_SMALLICON; | 36 size = SHGFI_SMALLICON; |
33 break; | 37 break; |
34 case IconLoader::NORMAL: | 38 case IconLoader::NORMAL: |
35 size = 0; | 39 size = 0; |
(...skipping 18 matching lines...) Expand all Loading... | |
54 image_skia.MakeThreadSafe(); | 58 image_skia.MakeThreadSafe(); |
55 image_.reset(new gfx::Image(image_skia)); | 59 image_.reset(new gfx::Image(image_skia)); |
56 DestroyIcon(file_info.hIcon); | 60 DestroyIcon(file_info.hIcon); |
57 } | 61 } |
58 } | 62 } |
59 | 63 |
60 // Always notify the delegate, regardless of success. | 64 // Always notify the delegate, regardless of success. |
61 target_message_loop_->PostTask(FROM_HERE, | 65 target_message_loop_->PostTask(FROM_HERE, |
62 base::Bind(&IconLoader::NotifyDelegate, this)); | 66 base::Bind(&IconLoader::NotifyDelegate, this)); |
63 } | 67 } |
OLD | NEW |