454247 - Download manager shows wrong icon for downloads if path to download folder contains non-ascii characters

Status: RESOLVED DUPLICATE of bug 415761

(Toolkit :: Downloads API, defect)

Description

[pal-moz]

Attachment: screenshot

range : 20080827-20080828 Nightly (Windows)

wrong icon appears, sometimes it's a folder icon, sometimes HDD drive icon.

this seems to happen when files are saved in 2-byte character name directory.

Comment 1

[memai]

Attachment: downloads window screenshot

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090408 Minefield/3.6a1pre

I can reproduce this bug.

load http://www.commeca.co.jp/president/images_p/%E7%9F%A5%E5%BA%8A.jpg
save to C:\Documents and Settings\username\My Documents\Downloads\
result:wrong icon(folder icon)

load http://www.mozilla.com/img/tignish/home/feature-logo.png
save to C:\Documents and Settings\username\My Documents\Downloads\
result:normal icon(Windows Picture and Fax Viewer icon)

load http://www.commeca.co.jp/president/images_p/%E7%9F%A5%E5%BA%8A.jpg
save to C:\Documents and Settings\username\My Documents\Downloads\ダウンロード\
result:wrong icon(folder icon)

load http://www.mozilla.com/img/tignish/home/feature-logo.png
save to C:\Documents and Settings\username\My Documents\Downloads\ダウンロード\
result:wrong icon(folder icon)

Comment 4

[Alexander L. Slovesnik]

Tweaking summary.

Comment 5

[Danil Ivanov]

normal: moz-icon://file:///C:/tmp/%D1%8B/somefile.png?size=32
wrong: moz-icon://file:///C:/tmp/somefile.png?size=32

Regression from Bug 422776?

Comment 6

[Danil Ivanov]

Sorry, I mean
wrong: moz-icon://file:///C:/tmp/%D1%8B/somefile.png?size=32
normal: moz-icon://file:///C:/tmp/somefile.png?size=32

Comment 7

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Can you reproduce it on trunk? (really anytime after http://hg.mozilla.org/mozilla-central/rev/3b4b7d84ee13)

Comment 8

[pal-moz]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090630 Minefield/3.6a1pre ID:20090630044527

seems to be fine.

Comment 9

[Masatoshi Kimura [:emk]]

Resolving per comment #8.