1199593 - [Message]The text content lost when forward the MMS.

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::SMS, defect)

Description

[Shine(Leave from Mozilla)]

Attachment: AriesKK2.5_video1.3gp

[1.Description]:
[Aries KK v2.5][Flame KK v2.5][Message]Receive a MMS from another phone that contains image (or music, video, vcf) and text content, launch Messages app and forward it, the text content lost when Forward this MMS.
See attachment: logcat_AriesKK2.5_1358.txt & AriesKK2.5_video1.3gp
Found time: 13:58

[2.Testing Steps]: 
1. Receive or send a MMS that contains image (or music, video, vcf) and text content. 
2. Launch Messages app and view the MMS.
3. Long tap the MMS and select"Forward". 
4. Check all the text from the news article in the message box. 

[3.Expected Result]: 
3. All content from the MMS should be retained in the message.

[4.Actual Result]: 
4. Only the image (or music, video, vcf) can be retained, the text content lost.

[5.Reproduction build]: 
Device: Aries KK v2.5 (Affected)
Build ID               20150827170125
Gaia Revision          d784c81961d82cbe9e111405468c590a8345856c
Gaia Date              2015-08-27 12:29:28
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/ca086f9ef8bca2d6cdfa79bfc4c854f56a59859e
Gecko Version          43.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20150827.162303
Firmware Date          Thu Aug 27 16:23:10 UTC 2015
Bootloader             s1

Device: Flame KK v2.5 (Affected)
Build ID               20150827150209
Gaia Revision          d784c81961d82cbe9e111405468c590a8345856c
Gaia Date              2015-08-27 12:29:28
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/ca086f9ef8bca2d6cdfa79bfc4c854f56a59859e
Gecko Version          43.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150827.190412
Firmware Date          Thu Aug 27 19:04:26 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device: Flame KK v2.2 (Unaffected)
Build ID               20150827032504
Gaia Revision          335cd8e79c20f8d8e93a6efc9b97cc0ec17b5a46
Gaia Date              2015-08-14 19:06:41
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/16d864d163de
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150827.064425
Firmware Date          Thu Aug 27 06:44:37 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

[6.Reproduction Frequency]: 
Always Recurrence,5/5

[7.TCID]: 
13323

Comment 1

[Shine(Leave from Mozilla)]

Attachment: logcat_AriesKK2.5_1358.txt

Comment 2

[Julien Wajsberg [:julienw]]

This is bad.

Can we please have a regression window ?
A guess would be the navigation patch from bug 1162030. But I thought I tested this case.

Comment 3

[Julien Wajsberg [:julienw]]

Found the issue, it's actually been broken by bug 1162030.

Comment 4

[GitHub Autolander Bot]

Attachment: [gaia] julienw:1199593-forward-text-attachment > mozilla-b2g:master

Comment 5

[Julien Wajsberg [:julienw]]

Comment on attachment 8654060 [details] [review]
[gaia] julienw:1199593-forward-text-attachment > mozilla-b2g:master

Hey Oleg,

is it OK for you if we don't do marionette tests for now as you're working on making storages work for MMS ?

I can file a separate bug to do a marionette test for forwarding a MMS if you want.

Comment 6

[Mahendra Potharaju [:mahe]]

Loss of information. Blocking 2.5

Comment 7

[Julien Wajsberg [:julienw]]

Comment on attachment 8654060 [details] [review]
[gaia] julienw:1199593-forward-text-attachment > mozilla-b2g:master

As agreed on IRC, working on integration tests now.

Comment 8

[Julien Wajsberg [:julienw]]

Comment on attachment 8654060 [details] [review]
[gaia] julienw:1199593-forward-text-attachment > mozilla-b2g:master

This time it's ready ! I hope you'll have time to look at it today :)

Comment 9

[Oleg Zasypkin [:azasypkin]]

Comment on attachment 8654060 [details] [review]
[gaia] julienw:1199593-forward-text-attachment > mozilla-b2g:master

Looks good! Just some minor nits and question at github.

Thanks!

Comment 10

[Julien Wajsberg [:julienw]]

master: b75979ec8862bd5799a7c42e938d3f67be38d6ae

Comment 11

[Martin Shuman [:Marty]]

This issue is verified fixed on the latest Flame and Aries 2.5 Master builds.
Both Media and Text content are properly included in the new, forwarded message.

Environmental Variables:
Device: Flame 2.5
BuildID: 20150903030230
Gaia: 29f363d6236bf7db8141d7a1f1185a1dcd809bf7
Gecko: a6786bf8d71d4cf40c3d40e06d8e3c9866863475
Gonk: c4779d6da0f85894b1f78f0351b43f2949e8decd
Version: 43.0a1 (2.5) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0

Environmental Variables:
Device: Aries 2.5
BuildID: 20150902124951
Gaia: e2fab8f6ac345ecde10a1350e699be9ceb6987d6
Gecko: 1b687fcb5213153855c7ac0f8392ce0a4a7e3382
Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd
Version: 43.0a1 (2.5) 
Firmware Version: D5803_23.1.A.1.28_NCB.ftf
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0