diff options
| author | kinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-19 23:21:58 +0000 |
|---|---|---|
| committer | kinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-19 23:21:58 +0000 |
| commit | f2ba4e66c27fe30a9575db9816458fb497bd6f29 (patch) | |
| tree | 4346a4c284161ec996238a2e62149e574feae908 /chrome/browser/gtk/download_item_gtk.h | |
| parent | 89c452823997708a50f11730d94615e4e701a003 (diff) | |
| download | chromium_src-f2ba4e66c27fe30a9575db9816458fb497bd6f29.zip chromium_src-f2ba4e66c27fe30a9575db9816458fb497bd6f29.tar.gz chromium_src-f2ba4e66c27fe30a9575db9816458fb497bd6f29.tar.bz2 | |
REGRESSION: Show correct exe icon in the downloadshelf
The bug has been introduced in r65418.
In r65418 I replaced some DownloadItem::full_path() with DownloadItem::GetUserVerifiedFileName() for loading icons, but on Windows it needed to be a full-path (rather than a file name).
BUG=62495
TEST=download mini_installer.exe and see if the correct icon is shown in the downloadshelf after the download is finished.
TEST=download regular files and see if the correct icon is shown (from the beginning).
Review URL: http://codereview.chromium.org/5147006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66838 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/gtk/download_item_gtk.h')
| -rw-r--r-- | chrome/browser/gtk/download_item_gtk.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/gtk/download_item_gtk.h b/chrome/browser/gtk/download_item_gtk.h index a9ad27d..8d79080 100644 --- a/chrome/browser/gtk/download_item_gtk.h +++ b/chrome/browser/gtk/download_item_gtk.h @@ -202,8 +202,8 @@ class DownloadItemGtk : public DownloadItem::Observer, SkBitmap* icon_small_; SkBitmap* icon_large_; - // The last download file name for which we requested an icon. - FilePath icon_filename_; + // The last download file path for which we requested an icon. + FilePath icon_filepath_; NotificationRegistrar registrar_; |