Closed
Bug 917167
Opened 11 years ago
Closed 11 years ago
[Buri][T-mobile 51612][MMS]The device cannot send MMS
Categories
(Firefox OS Graveyard :: RIL, defect, P1)
Firefox OS Graveyard
RIL
Tracking
(blocking-b2g:leo+, firefox25 wontfix, firefox26 fixed, firefox27 fixed, b2g18 fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 wontfix, b2g-v1.1hd fixed, b2g-v1.2 fixed)
People
(Reporter: sync-1, Assigned: airpingu)
References
Details
Attachments
(5 files)
Mozilla build ID:20130902041201
DEFECT DESCRIPTION:
The device cannot send MMS. There is always a failure message "Service currently unavailable. Message will automatically be sent once service is available"
MMS APN Settings seem correct. I cannot find a solution.
However the device can receive MMS
Customer Impact Statement:
Customer will not have this basic functionality of Smartphones. Rather poor impression of this new OS.
REPRODUCING PROCEDURES:
EXPECTED BEHAVIOUR:
ASSOCIATE SPECIFICATION:
TEST PLAN REFERENCE:
TOOLS AND PLATFORMS USED:
USER IMPACT:
REPRODUCING RATE:
For FT PR, Please list reference mobile's behavior:
Comment 8•11 years ago
|
||
Can you confirm this works with the same SIM on another device ?
Flags: needinfo?(sync-1)
Comment 9•11 years ago
|
||
(In reply to Julien Wajsberg [:julienw] from comment #8)
> Can you confirm this works with the same SIM on another device ?
Yes, the same SIM card can send MMS on android handset.
The most possible to reproduce this issue steps are:
Flash the handset -> FF OS start -> send the MMS, MMS can not be sent.
Updated•11 years ago
|
Flags: needinfo?(sync-1)
Comment 10•11 years ago
|
||
Can you compare the settings you have on android and the settings you have on the handset ? APN, but also MMSC proxy ?
Updated•11 years ago
|
Flags: needinfo?(sync-1)
Comment 11•11 years ago
|
||
When I wait a long time after flash FF OS on the device, the signal become normal, then I can send MMS success.
But sometimes it can not send MMS ok in this situation:
Compose a new MMS -> Select attachment -> Select Camera -> take a picture -> Insert
it in the MMS -> Try to send it.
The problem is reproducible only if you select to take a picture using camera while composing an MMS.
If the picture is selected from Gallery, the MMS can send.
Comment 12•11 years ago
|
||
I'm not sure this is the same problem.
About the first problem: I've noticed that the statusbar icon shows the signal maybe too soon, before we can actually use the network. Is it possible that this is the reason ? What do you mean by "a long time" ? 1 minute ? 2 minutes ? 5 minutes ?
About the second problem: I'll test it on my end.
Flags: needinfo?(felash)
Comment 13•11 years ago
|
||
android:
APN:3gwap
MMSC: http://mmsc.myuni.com.cn
MMS proxy: 10.0.0.172
MMS port: 80
FF OS:
APN:3gwap
MMS proxy: 10.0.0.172
MMS port: 80
MMSC: http://mmsc.myuni.com.cn
The setting is the same as I saw.
Flags: needinfo?(sync-1)
Comment 14•11 years ago
|
||
When sending MMS, change to airmode, MMS is sending, and then disable the airmode, MMS is sending all the time(circle progress is showing), but not OK any more.
Maybe it should be added some protect to avoid this situation. I think the first start device after flash is the same cause. The data network is not ok for send MMS.
Comment 15•11 years ago
|
||
I can't reproduce the picture problem.
Needinfo Gene for the sending problem. I think we fixed most of these problems recently. Gene, were they uplifted in 1.1 ?
Flags: needinfo?(felash) → needinfo?(gene.lian)
Comment 16•11 years ago
|
||
(In reply to Julien Wajsberg [:julienw] from comment #15)
> I can't reproduce the picture problem.
>
> Needinfo Gene for the sending problem. I think we fixed most of these
> problems recently. Gene, were they uplifted in 1.1 ?
Hi Julien Wajsberg,
What part did you modified, RIL? Gecko? Thanks!
Comment 17•11 years ago
|
||
Gene will know better than me ;)
Assignee | ||
Comment 18•11 years ago
|
||
Bug 901992 solved bugs for the MMS retrying mechanism. It will end up correctly turning into exclamation mark if the MMS failed to send after 4 times of retries. I'd suggest uplifting that.
Flags: needinfo?(gene.lian)
Comment 19•11 years ago
|
||
Are you saying that based on the steps on comment#14, this is a blocker for you guys ?
Flags: needinfo?(longxiuping)
Comment 20•11 years ago
|
||
(In reply to bhavana bajaj [:bajaj] from comment #19)
> Are you saying that based on the steps on comment#14, this is a blocker for
> you guys ?
Yes, device can not send MMS is a block.
But the original bug for MMS can not send is the following:
The device cannot send MMS. There is always a failure message "Service currently unavailable. Message will automatically be sent once service is available".
MMS APN Settings seem correct. I cannot find a solution.
However the device can receive MMS.
Flags: needinfo?(longxiuping)
Comment 21•11 years ago
|
||
Reproduce steps:
Compose a new MMS -> Select attachment -> Select Camera -> take a picture -> Insert it in the MMS -> Try to send it.
The problem is reproducible only if you select to take a picture using camera while composing an MMS.
If the picture is selected from Gallery, the MMS can send.
Comment 22•11 years ago
|
||
The symptoms in comment 20 look a lot like I describe in bug 909080.
Xiupinglong> can you send/receive SMS work in your case ? (in bug 909080 nothing was working: SMS, MMS, Data, Dial, but I could receive calls and SMS)
Comment 23•11 years ago
|
||
Broken (or unreliable) new 1.1 feature (MMS) = blocker. (We decided during triage that whether this is caused by bug 901992 or bug 909080 or is distinct is unimportant.)
blocking-b2g: leo? → leo+
Comment 24•11 years ago
|
||
I couldn't send MMS on my 1.1 unagi today, but maybe my connection was quite bad, I'll try again later.
Comment 25•11 years ago
|
||
Dear All,
I think there have some errors in refCount(In MmsService.js), please help to confirm it.
1. Send the first mms A , the refCount is zero in function acquire, if another mms B is sent before A send successful, the refCount will up to 1, then after A send successful, the refCount will decrease to zero in the function release, and there will set a delay to release MMS network (onDisconnectTimerTimeout), but mms B still not send successful (I think it's an error). If the onDisconnectTimerTimeout is called before B send success, here will occur an error, the second MMS unable send successful.
2. Send a mms A successful, send another mms B before onDisconnectTimerTimeout is called, if onDisconnectTimerTimeout is called before the mms B send successfull, then will occur an error, the mms B unable send successful. (I think it's an error that in onDisconnectTimerTimeout not judge refCount)
3. Continuously send mms, if some error occur in XHR, in xhr.onerror will call releaseMmsConnectionAndCallback, and in xhr.onreadystatechange will also call
releaseMmsConnectionAndCallback, then refCount will be reduce twice, it's an error.
Comment 26•11 years ago
|
||
(In reply to Julien Wajsberg [:julienw] from comment #22)
> The symptoms in comment 20 look a lot like I describe in bug 909080.
>
> Xiupinglong> can you send/receive SMS work in your case ? (in bug 909080
> nothing was working: SMS, MMS, Data, Dial, but I could receive calls and SMS)
Yes, sometimes device can send/receive SMS but can not send MMS.
Assignee | ||
Comment 27•11 years ago
|
||
(In reply to xiupinglong from comment #21)
> Reproduce steps:
> Compose a new MMS -> Select attachment -> Select Camera -> take a picture ->
> Insert it in the MMS -> Try to send it.
> The problem is reproducible only if you select to take a picture using
> camera while composing an MMS.
> If the picture is selected from Gallery, the MMS can send.
With the latest V1.1 (b2g18 build), I tried at least 3 times to send MMS from either Gallery or Camera respectively. Each of them is working well.
Hi, xiupinglong, could you please give it a try again or provide a concrete set of steps to reproduce? Asking for QA's support as well.
Keywords: qawanted
Comment 28•11 years ago
|
||
(In reply to Gene Lian [:gene] (national holidays until 9/23) from comment #27)
> Hi, xiupinglong, could you please give it a try again or provide a concrete
> set of steps to reproduce? Asking for QA's support as well.
Hi Gene Lian, could you see the comment #25, is the analysis right?
Assignee | ||
Comment 29•11 years ago
|
||
(In reply to Guoqiang.CHEN from comment #25)
> Dear All,
>
> I think there have some errors in refCount(In MmsService.js), please help to
> confirm it.
>
> 1. Send the first mms A , the refCount is zero in function acquire, if
> another mms B is sent before A send successful, the refCount will up to 1,
If B is sending before A is successfully sent, the refCount will up to 2 since B has already acquired one count?
> then after A send successful, the refCount will decrease to zero in the
> function release, and there will set a delay to release MMS network
> (onDisconnectTimerTimeout), but mms B still not send successful (I think
> it's an error). If the onDisconnectTimerTimeout is called before B send
> success, here will occur an error, the second MMS unable send successful.
>
> 2. Send a mms A successful, send another mms B before
> onDisconnectTimerTimeout is called, if onDisconnectTimerTimeout is called
> before the mms B send successfull, then will occur an error, the mms B
> unable send successful. (I think it's an error that in
> onDisconnectTimerTimeout not judge refCount)
You are correct! Nice catch! :) Or we should call disconnectTimer.cancel() whenever calling .acquire(...).
>
> 3. Continuously send mms, if some error occur in XHR, in xhr.onerror will
> call releaseMmsConnectionAndCallback, and in xhr.onreadystatechange will
> also call
> releaseMmsConnectionAndCallback, then refCount will be reduce twice, it's an
> error.
If the .onreadystatechange would always be executed even if .onerror has been executed, then this bug is valid. We might need to remove .onerror.
-----
Hi Guoqiang.CHEN,
I don't think these refCount bugs are the direct cause of this bug. Could you please fire separate bugs for them? That will be wonderful if you could please take them as well. If not, we can find someone fixing them.
Assignee | ||
Comment 30•11 years ago
|
||
(In reply to xiupinglong from comment #28)
> (In reply to Gene Lian [:gene] (national holidays until 9/23) from comment
> #27)
> > Hi, xiupinglong, could you please give it a try again or provide a concrete
> > set of steps to reproduce? Asking for QA's support as well.
>
> Hi Gene Lian, could you see the comment #25, is the analysis right?
Sounds right but the refCount related bugs only happens when sending MMS consecutively, which is not a symptom of this bug. Anyway, I cannot reproduce this bug on V1.1.
Comment 32•11 years ago
|
||
Dear Gene,
I create another PR 919440, I'm sorry I can't take this because I'm not very professional in MMS send.
>If B is sending before A is successfully sent, the refCount will up to 2 since B has already acquired one count?
When sending A, the refCount is zero(if not connected the acquire will not add refCount), when sending B the refCount will up to 1.
Flags: needinfo?(Chenguoqiang)
Assignee | ||
Comment 33•11 years ago
|
||
(In reply to Guoqiang.CHEN from comment #32)
> >If B is sending before A is successfully sent, the refCount will up to 2 since B has already acquired one count?
> When sending A, the refCount is zero(if not connected the acquire will not
> add refCount), when sending B the refCount will up to 1.
OK! I see. It's indeed a bug. Thanks for catching this! :)
Assignee | ||
Comment 34•11 years ago
|
||
Hi guys,
This patch fixes bug 917167, #c25. However, this bug isn't reproducing to me even before this patch.
This refCount issues do cause the failure of sending MMS under some circumstances, so I'll still prefer taking use of this bug to land it.
We're in the leo+ releasing cycle. Need more reviewers' eyes on this patch fix to avoid regression.
Attachment #808497 -
Flags: review?(vyang)
Attachment #808497 -
Flags: review?(ctai)
Attachment #808497 -
Flags: review?(Chenguoqiang)
Assignee | ||
Comment 35•11 years ago
|
||
Comment 36•11 years ago
|
||
Hi Gene Lian,
Device can not send MMS reproduce steps:
1. system language is Greek
>>>Compose a new MMS -> Select attachment -> Select Camera -> take a picture -> Insert it in the MMS -> Try to send it -> the MMS can not send.
>>>Compose a new MMS -> Select gallery -> select a picture which name is Greek language -> Insert it in the MMS -> Try to send it -> the MMS can not send.
2. system language is English
>>>Compose a new MMS -> Select gallery -> select a picture which name is Greek language -> Insert it in the MMS -> Try to send it -> the MMS can not send.
I will fire another bug for this.
Comment 37•11 years ago
|
||
Comment on attachment 808497 [details] [diff] [review]
Patch
Review of attachment 808497 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/mobilemessage/src/gonk/MmsService.js
@@ +557,2 @@
> xhr.onreadystatechange = function () {
> if (xhr.readyState != Ci.nsIXMLHttpRequest.DONE) {
Should we put releaseMmsConnectionAndCallback(0, null); right here?
Attachment #808497 -
Flags: review?(ctai)
Assignee | ||
Comment 38•11 years ago
|
||
(In reply to Chia-hung Tai [:ctai :ctai_mozilla :cht] from comment #37)
> Comment on attachment 808497 [details] [diff] [review]
> Patch
>
> Review of attachment 808497 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> ::: dom/mobilemessage/src/gonk/MmsService.js
> @@ +557,2 @@
> > xhr.onreadystatechange = function () {
> > if (xhr.readyState != Ci.nsIXMLHttpRequest.DONE) {
>
> Should we put releaseMmsConnectionAndCallback(0, null); right here?
No. Please see [1]. The readyState implies the state flow of loading page. We need to bail out all the states until DONE.
[1] https://developer.mozilla.org/en-US/docs/Web/API/XMLHttpRequest#Properties
Updated•11 years ago
|
Flags: needinfo?(gene.lian)
Assignee | ||
Comment 39•11 years ago
|
||
(In reply to xiupinglong from comment #36)
> Hi Gene Lian,
>
> Device can not send MMS reproduce steps:
> 1. system language is Greek
> >>>Compose a new MMS -> Select attachment -> Select Camera -> take a picture -> Insert it in the MMS -> Try to send it -> the MMS can not send.
>
> >>>Compose a new MMS -> Select gallery -> select a picture which name is Greek language -> Insert it in the MMS -> Try to send it -> the MMS can not send.
>
> 2. system language is English
> >>>Compose a new MMS -> Select gallery -> select a picture which name is Greek language -> Insert it in the MMS -> Try to send it -> the MMS can not send.
>
> I will fire another bug for this.
Awesome! Thanks for reproducing with better steps. I can look into that when you're done opening that bug (should be an leo+ one for any failure of sending MMS). I'll keep using this bug to solve the refCount issue which is causing another kind of failure of sending MMS.
Flags: needinfo?(gene.lian)
Comment 40•11 years ago
|
||
I fire another bug 919465 for comment 36.
Comment 41•11 years ago
|
||
Comment on attachment 808497 [details] [diff] [review]
Patch
Review of attachment 808497 [details] [diff] [review]:
-----------------------------------------------------------------
According to comment 38, looks good to me.
Attachment #808497 -
Flags: review+
Comment 42•11 years ago
|
||
Comment on attachment 808497 [details] [diff] [review]
Patch
Review of attachment 808497 [details] [diff] [review]:
-----------------------------------------------------------------
I verify it is ok, thanks.
Attachment #808497 -
Flags: review?(Chenguoqiang) → review+
Assignee | ||
Comment 43•11 years ago
|
||
Ping Vicamo for the review. This is leo+. I'm afraid we need to move fast. I'd suggest let's focus on solving bug first. If you have any better insight about the architecture, we can keep polishing it in koi or 1.3. ;)
Assignee: nobody → gene.lian
Updated•11 years ago
|
Component: Gaia::SMS → RIL
Comment 44•11 years ago
|
||
Comment on attachment 808497 [details] [diff] [review]
Patch
Review of attachment 808497 [details] [diff] [review]:
-----------------------------------------------------------------
Thank you. I've also filed an follow-up bug 920362 for test cases of mms data connection acquisition/release.
Attachment #808497 -
Flags: review?(vyang) → review+
Assignee | ||
Comment 45•11 years ago
|
||
Assignee | ||
Comment 46•11 years ago
|
||
The patch for b2g18 is also ready in the attachment list. Please help uplift that when it lands to central. Thanks!
status-b2g18:
--- → affected
status-b2g18-v1.0.0:
--- → wontfix
status-b2g18-v1.0.1:
--- → wontfix
status-b2g-v1.1hd:
--- → affected
status-b2g-v1.2:
--- → affected
Comment 47•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 48•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/b357c5acdc06
https://hg.mozilla.org/releases/mozilla-b2g18/rev/6f04a111aeb1
Comment 49•11 years ago
|
||
status-firefox25:
--- → wontfix
You need to log in
before you can comment on or make changes to this bug.
Description
•