Closed
Bug 1069793
Opened 10 years ago
Closed 10 years ago
[Loop] While GSM call is established, Loop call can't be established
Categories
(Firefox OS Graveyard :: Gaia::Loop, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mbarone976, Assigned: jaoo)
References
Details
(Whiteboard: [mobile app][blocking][tef-triage][patch available])
Attachments
(4 files)
Device: Flame
Build: v2.0 with kk v180 (gecko-2d2ca12 gaia-7edd3b0)
Loop version: c28eaef
STR
1. Establish a GSM call between two devices (Device A and Device B). Device A calls Device B
2. From a device C, try to establish a loop call with Device B
ACTUAL RESULT
The attention screen shown is wrong and is not possible to answer the loop call (See attachment)
Updated•10 years ago
|
OS: Mac OS X → Gonk (Firefox OS)
Hardware: x86 → ARM
Updated•10 years ago
|
Whiteboard: [mobile app][blocking] → [mobile app][blocking][tef-triage]
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → josea.olivera
Assignee | ||
Comment 1•10 years ago
|
||
Fix the root cause of this bug on bug 1069847 (FYI pretty related root cause).
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 2•10 years ago
|
||
Tested again, and it's still happening.
Device: Flame
Build: v2.0 with kk v180 (Gecko-9362b48.Gaia-263e3b2)
Loop version: 041ff17
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 3•10 years ago
|
||
Waiting some seconds (to answer de call), or if GSM call is on hold, It works fine.
Comment 4•10 years ago
|
||
....
E/GeckoConsole( 3286): Content JS LOG at app://loop.services.mozilla.com/call_screen/js/call_screen_ui.js:83 in CallScreenUI.init/<: An error occured! Permission Denied
...
Assignee | ||
Comment 5•10 years ago
|
||
(In reply to Javier de Prado from comment #4)
> Created attachment 8494423 [details]
> bug1069793.txt
>
> ....
> E/GeckoConsole( 3286): Content JS LOG at
> app://loop.services.mozilla.com/call_screen/js/call_screen_ui.js:83 in
> CallScreenUI.init/<: An error occured! Permission Denied
> ...
Actually the issue is because a race condition while setting the call on hold. The message shown here, when failing at gUM, might not be correct (as it is happening in here).
Assignee | ||
Comment 6•10 years ago
|
||
The wrong call screen thing is fixed but we need to figure out why the gUM request fail when the GSM call is up. Change the bug name then.
Summary: [Loop] During a GSM call, if an incoming loop call is received, the attention screen shows call ended → [Loop] While GSM call is established, Loop call can't be established
Comment 7•10 years ago
|
||
and sometimes it appears and disappears this screen asking for the permissions
Assignee | ||
Comment 8•10 years ago
|
||
gUM fails with 'Permission Denied' error when It shouldn't as all the permission are granted. I've have seen the error is because a race condition as a 1-second-timeout fix solves the issue. Here you have that patch. Let's discuss a bit this and other possible solutions.
Attachment #8494655 -
Flags: feedback?(borja.bugzilla)
Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8494655 [details]
Pointer to Github PR https://github.com/mozilla-b2g/firefoxos-loop-client/pull/131
Borja, I ended up with a solution which is to set the GSM call on hold earlier (by competing for audio) so gUM doesn't fail anymore. Have a look please. Thanks!
Attachment #8494655 -
Flags: feedback?(borja.bugzilla) → review?(borja.bugzilla)
Comment 10•10 years ago
|
||
Comment on attachment 8494655 [details]
Pointer to Github PR https://github.com/mozilla-b2g/firefoxos-loop-client/pull/131
Main thing is that it seems that there is no async. request in your code, so we should get rid of the promise.
Take a look to the comments in GH and ask me to review this again when ready. Thanks!
Attachment #8494655 -
Flags: review?(borja.bugzilla)
Assignee | ||
Comment 11•10 years ago
|
||
Comment on attachment 8494655 [details]
Pointer to Github PR https://github.com/mozilla-b2g/firefoxos-loop-client/pull/131
(In reply to Borja Salguero [:borjasalguero] from comment #10)
> Comment on attachment 8494655 [details]
> Pointer to Github PR
> https://github.com/mozilla-b2g/firefoxos-loop-client/pull/131
>
> Main thing is that it seems that there is no async. request in your code, so
> we should get rid of the promise.
Forget that patch please.
> Take a look to the comments in GH and ask me to review this again when
> ready. Thanks!
Done, I took your suggestion and gave it a try. Seems to work.
Javier, could you give a try and let us know if the patch fixes the issue on your side as well please? A double-check would be great.
Attachment #8494655 -
Flags: review?(borja.bugzilla)
Attachment #8494655 -
Flags: feedback?(javier.deprado)
Updated•10 years ago
|
Whiteboard: [mobile app][blocking][tef-triage] → [mobile app][blocking][tef-triage][patch available]
Assignee | ||
Comment 12•10 years ago
|
||
Comment on attachment 8494655 [details]
Pointer to Github PR https://github.com/mozilla-b2g/firefoxos-loop-client/pull/131
Borja, this is finally fixed. Both incoming and outgoings call. It was a matter of setting the GSM call on hold as earlier as possible. Now gUM doesn't fail and everything seems to work.
Javier, just verify the patch when it lands.
Attachment #8494655 -
Flags: feedback?(javier.deprado)
Comment 13•10 years ago
|
||
Comment on attachment 8494655 [details]
Pointer to Github PR https://github.com/mozilla-b2g/firefoxos-loop-client/pull/131
Please rebase and remove comments before merging. Thanks!
Attachment #8494655 -
Flags: review?(borja.bugzilla) → review+
Assignee | ||
Comment 14•10 years ago
|
||
Landed at:
https://github.com/mozilla-b2g/firefoxos-loop-client/commit/47ad47ff50be6838fc9511093317c65062ea8bf2
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Comment 15•10 years ago
|
||
Loop Version 1.1: e4ffbba
FireE: firee-kk-v2.0-SW2E5-4
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 16•10 years ago
|
||
Closing this bug as this issue was already resolved and new bug 1091790, with the specific problem was opened to track the issue found in the regression test.
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•