[tarako] unable to add new group call after Communication app is killed

RESOLVED FIXED

Status

defect
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: angelc04, Assigned: etienne)

Tracking

({verifyme})

unspecified
Other
Other

Firefox Tracking Flags

(b2g-v1.3T fixed)

Details

(Whiteboard: [sprd313255][tarako-only])

Attachments

(2 attachments, 1 obsolete attachment)

Posted file slog.zip
Steps to reproduce
--------------------------------------------------------------------
1. Make a call and establish the call connection
2. Use adb shell to kill Communication app (this is to simulate the case that Communication is killed by OOM/LMK)
3. Try to add a new call on device
   --> The second call was always ended automatically.

Please see attached slog.

Test build
---------------------------------------------------------------------------
Gaia      ffa388c3c00ee7a31216b397acaa07e1cf94bb39
Gecko     https://hg.mozilla.org/releases/mozilla-b2g28_v1_3t/rev/79eb01b64ed6
BuildID   20140525164003
Version   28.1
ro.build.version.incremental=eng.cltbld.20140520.195709
ro.build.date=Tue May 20 19:57:15 EDT 2014
Flags: needinfo?(szchen)
Hi Aknow:
 Please help check this issue first per offline talk.

Thanks!!
Shawn
01-01 00:15:03.180    85    85 I Gecko   : TelephonyProvider: Dialing 13312181721

01-01 00:15:03.570    96   350 D RILC    : [0086]> DIAL (num=13312181721,clir=0)
01-01 00:15:03.590    96   350 D RILC    : [0086]< DIAL
01-01 00:15:04.030    96   349 D RILC    : [0087]< GET_CURRENT_CALLS {[id=1,ACTIVE,toa=161,norm,mt,als=0,voc,noevp,02284841399,cli=0,name='(null)',0}

01-01 00:15:04.180    85    85 I Gecko   : TelephonyProvider: handleCallDisconnected: {"number":"13312181721","state":2,"callIndex":4294967295,"isOutgoing":true,"isEmergency":false,"isConference":false,"failCause":"UnspecifiedError"}
blocking-b2g: --- → 1.3T?
Whiteboard: [sprd313255]
In gaia, we have to make sure the telephony object is fully initialized; then we could perform the operations on it or read its attribute value. The completion of initializing is synchronized by 1st 'callschanged' event. So in the case of this issue, we should ensure that the event is received before calling telephony.dial().

attachment 8429148 [details] [diff] [review] shows a sample modification based on this idea and I have verified it.


Etienne,

Could you take this bug and provide a formal patch on gaia? Thank you.
Flags: needinfo?(etienne)
(In reply to Szu-Yu Chen [:aknow] from comment #4)
> In gaia, we have to make sure the telephony object is fully initialized;
> then we could perform the operations on it or read its attribute value. The
> completion of initializing is synchronized by 1st 'callschanged' event. So
> in the case of this issue, we should ensure that the event is received
> before calling telephony.dial().
> 
> attachment 8429148 [details] [diff] [review] shows a sample modification
> based on this idea and I have verified it.
> 
> 
> Etienne,
> 
> Could you take this bug and provide a formal patch on gaia? Thank you.

Yes I can provide a 1.3t patch but I really think there should be a gecko patch for this on trunk if the issue happens elsewhere.
Flags: needinfo?(etienne)
Assignee: nobody → etienne
Whiteboard: [sprd313255] → [sprd313255][tarako-only]
?w=1 recommended
Attachment #8429148 - Attachment is obsolete: true
Attachment #8429294 - Flags: review?(anthony)
Comment on attachment 8429294 [details]
Gaia PR against 1.3t

>https://github.com/mozilla-b2g/gaia/pull/19678/files
Attachment #8429294 - Attachment mime type: text/plain → text/x-github-pull-request
Attachment #8429294 - Attachment mime type: text/x-github-pull-request → text/plain
Comment on attachment 8429294 [details]
Gaia PR against 1.3t

Comments left on the PR.
Attachment #8429294 - Flags: review?(anthony)
(In reply to Etienne Segonzac (:etienne) from comment #5)
> (In reply to Szu-Yu Chen [:aknow] from comment #4)
> > In gaia, we have to make sure the telephony object is fully initialized;
> > then we could perform the operations on it or read its attribute value. The
> > completion of initializing is synchronized by 1st 'callschanged' event. So
> > in the case of this issue, we should ensure that the event is received
> > before calling telephony.dial().
> > 
> > attachment 8429148 [details] [diff] [review] shows a sample modification
> > based on this idea and I have verified it.
> > 
> > 
> > Etienne,
> > 
> > Could you take this bug and provide a formal patch on gaia? Thank you.
> 
> Yes I can provide a 1.3t patch but I really think there should be a gecko
> patch for this on trunk if the issue happens elsewhere.

You are right. We are thinking about how to return an already initialized object for navigator.mozTelephony.
Comment on attachment 8429294 [details]
Gaia PR against 1.3t

updated
Attachment #8429294 - Flags: review?(anthony)
Attachment #8429294 - Flags: review?(anthony) → review+
The patch is ready to go if we get 1.3t+/1.3t-approval, otherwise we can close this bug :)
blocking-b2g: 1.3T? → 1.3T+
Hi Etienne, Should we keep this bug in RIL component? I would like to file a bug for comment 10.
Flags: needinfo?(etienne)
(In reply to Ken Chang[:ken] from comment #13)
> Hi Etienne, Should we keep this bug in RIL component? I would like to file a
> bug for comment 10.

I think we should file a follow up for the RIL fix on central.
This one is already 1.3t+, marked tarako-only etc...
Flags: needinfo?(etienne)
(In reply to Etienne Segonzac (:etienne) from comment #14)
> (In reply to Ken Chang[:ken] from comment #13)
> > Hi Etienne, Should we keep this bug in RIL component? I would like to file a
> > bug for comment 10.
> 
> I think we should file a follow up for the RIL fix on central.
> This one is already 1.3t+, marked tarako-only etc...

Done, it is bug 1018045
https://github.com/mozilla-b2g/gaia/commit/1cd5f640a586a2f8c46af8f9de85b2792d511f1d
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
[Blocking Requested - why for this release]: According the discussions in bug1061210, this issue also exists in the branch v1.4 and we should land the patch to fix it.

Thanks a lot:)
blocking-b2g: 1.3T+ → 1.4?
Vance, you may want to triage this for 1.4.
Flags: needinfo?(vchen)
v1.4 is EOL
blocking-b2g: 1.4? → ---
Flags: needinfo?(vchen)
You need to log in before you can comment on or make changes to this bug.