889746 - [MMS-Gallery] Allow *.wbmp file to be stored via Device Storage API

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: General, defect, P1)

Description

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

Attachment: Add *.wbmp in devicestorage.properties

Saving a WBMP file received from MMS message will get a TypeMismatchError because *.wbmp is not in the while list of supported file extension in Device Storage. We'll need to add *.wbmp in the while list before we completely remove WBMP support from gecko in bug 882526.

Comment 1

[Steve Chung [:steveck]]

Hi Wayne, If we have to save the image to wbmp format instead of PNG(gaia decode solution), then this patch is must have. Could you update the information? Thanks.

Comment 2

[Andreas Gal :gal]

We can export at WBMP and read PNG and convert on the fly to WBMP to send it. Isn't that much easier?

Comment 3

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

(In reply to Andreas Gal :gal from comment #2)
> We can export at WBMP and read PNG and convert on the fly to WBMP to send
> it. Isn't that much easier?

Not sure if I fully understand your suggestion, currently we cannot save a WBMP file into device storage without this patch.

Or do you mean we save the WBMP as a PNG file with annotation, e.g. xxx.wbmp.png, and try covert this kind of PNG back to WBMP format when user want attach this image in an MMS message?

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

See https://bugzilla.mozilla.org/show_bug.cgi?id=882526#c12. So far I don't see why saving a WBMP to device storage is a must-have feature.

Comment 5

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

This is just an alternative approach if we cannot convince our partner to save WBMP as PNG file for bug 887164, and I agree not to support *.wbmp file since we are going to drop the WBMP support from gecko in a near future.

Comment 6

[Doug Turner (:dougt)]

Comment on attachment 770672 [details] [diff] [review]
Add *.wbmp in devicestorage.properties

per last comment.

Comment 7

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

As comment 4 and comment 5 said, we are not going to add *.wbmp as our supported image file because WBMP is not an open web standard. We'll convert WBMP as PNG image in Message app for viewing and saving.