224216 - Mozilla cant render images with .Jpg extension

Status: VERIFIED DUPLICATE of bug 223055

(MailNews Core :: Attachments, defect)

Description

[Marc Chamberlin]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20030925
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20030925

I suspect the capital J in the Jpg extension is confusing Mozilla. Try
normalizing to all lower case, the extensionon, before looking up the helper
application to handle the image.

Reproducible: Always

Steps to Reproduce:
1.Post a message with an image attachment to a newsgroup with a .Jpg extension
2.Open the message with Mozilla.
3.

Actual Results:  
Mozilla fails to find a helper application (though it knows it is a jpeg image type)

Expected Results:  
I expected the image to be rendered and displayed.

Comment 1

[Boris Zbarsky [:bzbarsky]]

biesi, the windows code probably needs to lowercase the ext at
http://lxr.mozilla.org/seamonkey/source/uriloader/exthandler/win/nsOSHelperAppService.cpp#334
(just call ToLowerCase on fileExtToUse after appending aFileExt).... what do you
think?

Alternately, we could handle the lowecasing at the nsExternalHelperAppService
level before calling GetMIMEInfoFromOS.. but the other OSHelperAppServices have
to deal with non-lowercase data coming from the OS too, so they're fully
case-insensitive already.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: Somewhat like that (needs testing)

Someone with a Windows system needs to test this.

Comment 3

[Christian :Biesinger (don't email me, ping me on IRC)]

Comment on attachment 134498 [details] [diff] [review]
Somewhat like that (needs testing)

<NeilAway> registry is case insensitive
(I wrote a simple test program which confirmed that)

reporter: could you give the url for an example page?

Comment 4

[Christian :Biesinger (don't email me, ping me on IRC)]

oh nevermind, news... so your real problem is that you get the helper app
dialog, and the page is not rendered? that may have been fixed by bz's recent
checkin to case-insensitively check extensions in GetTypeFromExtension

bz: note that the reporter was using linux :)

Comment 5

[Marc Chamberlin]

CB You have surmised the problem correctly. 

Comment 6

[Boris Zbarsky [:bzbarsky]]

Ah, yes.  On linux, that would be a problem... ;)  I don't know what I was
smoking....

*** This bug has been marked as a duplicate of 223055 ***

Comment 7

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified DUP.