842037 - Downloaded pictures from Firefox Mobile are incompatible with MMS-messaging on Android - Android not recognizing our saved media's Content-Type

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Eric]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:18.0) Gecko/20100101 Firefox/18.0
Build ID: 20130201065344

Steps to reproduce:

I downloaded pictures from various websites using the most recent version of Firefox Mobile Beta (as of writing this) on a Galaxy Nexus, running Android 4.2.1. I then tried to use said downloaded pictures in picture messages.


Actual results:

 I was unable to send picture messages that contained pictures downloaded using Firefox Mobile. I received an error message informing I was "unable to attach this photo" (I have redownloaded same pictures using stock 4.2.1 browser, and they are able to be attached).  This error has occurred across multiple Firefox versions-- I had decided to hold off on reporting it to see if it would resolve itself. It hasn't.


Expected results:

They should have been able to be attached to my picture messages.

Comment 1

[Eric]

I have listed the severity as major as I believe the inability to send downloaded pictures in messages from a Mobile browser that is primarily aimed at phones is a serious flaw.

Comment 2

[Aaron Train [:aaronmt]]

I can reproduce this on my LG Nexus 4 (Android 4.2.2), and my SII (Android 4.0.4).

As a work-around, you can opt to share the image (via long-press hold on a selected image, and selecting 'Share Image') and select your text-messaging application of choice.

Comment 3

[Eric]

(In reply to Aaron Train [:aaronmt] from comment #2)
> I can reproduce this on my LG Nexus 4 (Android 4.2.2), and my SII (Android
> 4.0.4).
> 
> As a work-around, you can opt to share the image (via long-press hold on a
> selected image, and selecting 'Share Image') and select your text-messaging
> application of choice.
Noted, will be taking use of that as a temporary work-around. It's become annoying as I prefer to use Firefox as my default browser. 

Thanks for the changes, never filed a bug report before.

Comment 4

[Aaron Train [:aaronmt]]

(In reply to Eric from comment #3)
> Thanks for the changes, never filed a bug report before.

No problem, thanks for filing!

Here is the error stack in console

E/WorkingMessage( 3769): com.google.android.mms.MmsException: Type of media is unknown.
E/WorkingMessage( 3769): 	at com.android.mms.model.ImageModel.initModelFromUri(ImageModel.java:90)
E/WorkingMessage( 3769): 	at com.android.mms.model.ImageModel.<init>(ImageModel.java:74)
E/WorkingMessage( 3769): 	at com.android.mms.ui.SlideshowEditor.changeImage(SlideshowEditor.java:163)
E/WorkingMessage( 3769): 	at com.android.mms.data.WorkingMessage.internalChangeMedia(WorkingMessage.java:640)
E/WorkingMessage( 3769): 	at com.android.mms.data.WorkingMessage.changeMedia(WorkingMessage.java:588)
E/WorkingMessage( 3769): 	at com.android.mms.data.WorkingMessage.setAttachment(WorkingMessage.java:453)
E/WorkingMessage( 3769): 	at com.android.mms.ui.ComposeMessageActivity.addImage(ComposeMessageActivity.java:3150)
E/WorkingMessage( 3769): 	at com.android.mms.ui.ComposeMessageActivity.access$5700(ComposeMessageActivity.java:167)
E/WorkingMessage( 3769): 	at com.android.mms.ui.ComposeMessageActivity$33.run(ComposeMessageActivity.java:3140)
E/WorkingMessage( 3769): 	at com.android.mms.ui.AsyncDialog$ModalDialogAsyncTask.doInBackground(AsyncDialog.java:129)
E/WorkingMessage( 3769): 	at com.android.mms.ui.AsyncDialog$ModalDialogAsyncTask.doInBackground(AsyncDialog.java:84)
E/WorkingMessage( 3769): 	at android.os.AsyncTask$2.call(AsyncTask.java:287)
E/WorkingMessage( 3769): 	at java.util.concurrent.FutureTask.run(FutureTask.java:234)
E/WorkingMessage( 3769): 	at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:230)
E/WorkingMessage( 3769): 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1080)
E/WorkingMessage( 3769): 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:573)
E/WorkingMessage( 3769): 	at java.lang.Thread.run(Thread.java:856)

http://grepcode.com/file/repository.grepcode.com/java/ext/com.google.android/android-apps/4.1.1_r1/com/android/mms/model/ImageModel.java?av=f#90

Looks like it's failing to get a proper ContentType

This would need to be debugged in ContentAreaUtils to see what the ContentType is 

http://mxr.mozilla.org/mozilla-central/source/toolkit/content/contentAreaUtils.js#106

Comment 5

[:Margaret Leibovic]

I was able to reproduce and I added some logging in saveImageURL. With the image I tested with, contentType was "image/jpeg", which seems to make sense. It will take some more digging to figure out where things are going wrong.

I reproduced the problem on my Galaxy Nexus (4.2.2), but I couldn't reproduce on my Atrix (2.3.6), so this looks like a Android version-specific bug.

Comment 6

[cbalogh8888]

I am currently having this issue on my Samsung Galaxy S3. Has any solution been found as of yet?

Comment 7

[Kevin Brosnan [Ex-Mozilla]]

From comment 2

"As a work-around, you can opt to share the image (via long-press hold on a selected image, and selecting 'Share Image') and select your text-messaging application of choice."

Comment 8

[cbalogh8888]

(In reply to Kevin Brosnan [:kbrosnan] from comment #7)
> From comment 2
> 
> "As a work-around, you can opt to share the image (via long-press hold on a
> selected image, and selecting 'Share Image') and select your text-messaging
> application of choice."

Yes, I am fully aware of that work-around. Thank you.

But I like to have images saved on my phone for continuous use. I was bumping this just to see if a 'fix' has been developed (or is in the works) since the previous post was in late February.

Comment 9

[Aaron Train [:aaronmt]]

Currently the mobile developers are not tracking this issue as important as it has been minused in triage. As it stands now, this bug is open for anyone interested in tackling and patches are welcome.

Comment 10

[Kevin Brosnan [Ex-Mozilla]]

This seems like a good candidate for a mentor bug. Margaret any suggestions on whom would make a good mentor?

Comment 11

[:Margaret Leibovic]

Maybe Wes?

Comment 12

[Wesley Johnston (:wesj)]

Attachment: Patch

Mentoring this was going to be tough, because we don't know the problem. Thinking about it made me realize the problem though! We're not setting the right mimetype for the media scanner. Its probably then storing that null mimetype and confusing the messaging app.

This tries harder to get a mimetype. Fixes this on my S3.

Comment 13

[Brad Lassey [:blassey] (use needinfo?)]

Comment on attachment 802525 [details] [diff] [review]
Patch

Review of attachment 802525 [details] [diff] [review]:
-----------------------------------------------------------------

Wes have you tested this? or is it speculative? I have seen MimeTypeMap.getMimeTypeFromExtension() return null for what should be relatively obvious extensions.

Comment 14

[Wesley Johnston (:wesj)]

I tested it on my S3. I did a Google Image search for "kittens". Long tapped. Save Image. Then tried to attach that image in the messaging app. Worked with the patch. Failed without.

Have a ref to where it failed?

http://androidxref.com/4.2.2_r1/xref/external/okhttp/src/main/java/libcore/net/MimeUtils.java#46

Comment 15

[Brad Lassey [:blassey] (use needinfo?)]

Comment on attachment 802525 [details] [diff] [review]
Patch

Review of attachment 802525 [details] [diff] [review]:
-----------------------------------------------------------------

Wes have you tested this? or is it speculative? I have seen MimeTypeMap.getMimeTypeFromExtension() return null for what should be relatively obvious extensions.

Comment 16

[Brad Lassey [:blassey] (use needinfo?)]

in bug 780379 we're seeing it not get a mimetype for .m3u8, but the source you linked to explains it (.m3u is in there, but .m3u8 isn't).

Comment 17

[Wesley Johnston (:wesj)]

https://hg.mozilla.org/integration/fx-team/rev/3d36387df33f

Comment 18

[Wesley Johnston (:wesj)]

Comment on attachment 802525 [details] [diff] [review]
Patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Old media scanner bug
User impact if declined: Can't attach fennec's downloaded images to mms messages
Testing completed (on m-c, etc.): Landed on mc today
Risk to taking this patch (and alternatives if risky): Low risk. Just using existing code to fixup our mimetype.
String or IDL/UUID changes made by this patch: None.

Comment 19

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/3d36387df33f

Comment 20

[Alex Keybl [:akeybl]]

Comment on attachment 802525 [details] [diff] [review]
Patch

Not a recent regression, but a low risk fix for something that really impacts the user experience for some users. Let's skip the trains.

Comment 21

[Wesley Johnston (:wesj)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/367a18db3d09

Comment 22

[Teodora Vermesan (:TeoVermesan)]

Verified fixed on:
Build: Firefox for Android 25.0b1 (2013-09-18)
Device: LG Nexus 4
OS: Android 4.2.2

Comment 23

[Teodora Vermesan (:TeoVermesan)]

Verified fixed on:
Build: Firefox for Android 26.0a2 (2013-09-18)
Device: LG Nexus 4
OS: Android 4.2.2