31090 - file: protocol does not check Macintoish file type/create type

Status: VERIFIED DUPLICATE of bug 31054

(Core :: Networking, defect, P3)

Description

[Frank Tang]

This is a generalized problem of bug 31054. We currently do not check Macintosh 
file type / creator type to map MIME type. I think you should find a Mac guy to 
help you in this area. I think the Mac file type should be part of the 
nsIMIMEService. If you try the Mac version of 4.x, look at the 
preference:Application, select some type (try GIF) and click "EDIT", you will 
see that there are a box called "File Info" , it list the file type if the 
system have those Macintosh file type. 

The mozilla classic source code also show it access InternetConfig to find out 
the mapping between mac file type (not extension) and content-type. 

A hacky way to fix this is to add code in 
326 NS_IMETHODIMP
327 nsFileChannel::GetContentType(char * *aContentType)

add some #ifdef XP_MAC code before line 350 and 

349         if (mContentType.IsEmpty()) {
350             mContentType = UNKNOWN_CONTENT_TYPE;
351         }
352     }

Comment 1

[Warren Harris]

Frank: Why is this any different from bug 31054? nsIMIMEService should be 
looking at Mac file types, and not just file extensions.

Giving this to David too.

Comment 2

[Warren Harris]

=> M15

Comment 3

[Frank Tang]

No big differences from 31054... I think 31054 is just a special case for this 
one.

You may want to also consder adding InternetConfig support sine the old code 
does that. It seems the nsIMIMEService should also delegate some code to 
platform specialized object. 

Comment 4

[Simon Fraser [no longer active]]

A note to the implemnetor (i.e. davidm). We also want Internet config support for 
bug 29594, so we might want to share common IC code, or perhaps enhance the 
nsIUserInfo interface to make MIME info available.

Comment 5

[Warren Harris]

Marking beta2.

Comment 6

[davidm]

marking as a duplicate.


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

Comment 7

[Tom Everingham]

verif.