919853 - Unable to get corresponding extension from nsIMIMEService::GetPrimaryExtension

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[Eric Chou [:ericchou] [:echou]]

In order to be capable of sharing contacts via Bluetooth while no SD card installed, Gecko Bluetooth has to deal with a 'blob' but not a 'file'. The main difference is: blob doesn't have 'name' property. However, most mobile devices would turn down the file sharing request once the file name is empty or the extension is missing.

Currently Gecko Bluetooth uses function nsIMIMEService::GetPrimaryExtension(), trying to get the primary extension from mime type. Take sharing contacts as an example, the mime type "text/vcard" is sent to function GetPrimaryExtension() as the first argument, but the out parameter won't get any value.

I have traced related parts of code, found that there is a table in nsExternalHelperAppService.cpp for mapping extension and mimetype. We should add text/vcard and .vcf mapping record in that table.

Comment 1

[Eric Chou [:ericchou] [:echou]]

Attachment: patch 1: v1: Add mimetype and extension mapping of vCard

This patch does:

1. Add an record of vCard mapping in table extraMimeEntries
2. Replace deprecated mime type definition "text/x-vcard" with "text/vcard"

So that we can get corresponding extension (.vcf) from vCard blobs which have mime type "text/vcard".

Hi bz,

I saw you leaving comments in bug 891334 about the extraMimeEntries change. My patch also changes that table. Could you assist with reviewing my patch?

Hi francisco,

According to RFC 6350, "text/x-vcard" should have been deprecated in favor of "text/vcard". That's why I modified the definition of TEXT_VCARD in nsMimeTypes.h. Please assist with reviewing my patch.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 808978 [details] [diff] [review]
patch 1: v1: Add mimetype and extension mapping of vCard

r=me

Comment 3

[Francisco Jordano [:arcturus] [:francisco]]

Comment on attachment 808978 [details] [diff] [review]
patch 1: v1: Add mimetype and extension mapping of vCard

Looking good to me, we will change the references in the frontend to use the correct mimetype.

Thanks!

Comment 4

[Eric Chou [:ericchou] [:echou]]

Thank you both :)

https://hg.mozilla.org/integration/b2g-inbound/rev/b16fd01c41f6

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b16fd01c41f6