Closed
Bug 820642
Opened 12 years ago
Closed 12 years ago
Call waiting incomingEnd button sends a hold for the active call first instead of ending the active call
Categories
(Firefox OS Graveyard :: Gaia::Dialer, defect, P1)
Tracking
(blocking-basecamp:+)
People
(Reporter: anshulj, Assigned: etienne)
Details
Attachments
(1 file, 1 obsolete file)
There is a GCF test case 31.3.1.2.1 that fails because during a call waiting scenario when ending the active call and accepting the incoming call the active call should be ended but Gaia is first sending the hold for the active call followed by the end for the active call. This is due to the fact that endAndAnswer method in oncall.js first calls holdAndAnswer and then calls callToEnd.hangup.
Updated•12 years ago
|
blocking-basecamp: --- → ?
Hsin-Yi, tentatively assigning to you. Please reassign as appropriate :).
Assignee: nobody → htsai
blocking-basecamp: ? → +
Priority: -- → P1
Updated•12 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 2•12 years ago
|
||
I investigated this issue and came to the same conclusion with Anshul (comment 0). I don't why in gaia/oncall.js we hold the first call then hang it up, instead of hanging it up directly. Maybe Etienne could help answer my question :) I would say it's a gaia issue so I reassign to Etienne. If we realize this issue should be fixed in gecko, then I'd love to take this!
Assignee: htsai → etienne
Component: General → Gaia::Dialer
Updated•12 years ago
|
Target Milestone: --- → B2G C3 (12dec-1jan)
Tested in the emulator and works, but I don't have the ability to test on hardware atm.
Assignee: etienne → jones.chris.g
Attachment #691685 -
Flags: review?(etienne)
Assignee | ||
Comment 4•12 years ago
|
||
Comment on attachment 691685 [details] [diff] [review] Hang up the current call and then answer the waiting call Not sure why, and didn't find an easy fix, but the second call never gets picked up...
Attachment #691685 -
Flags: review?(etienne) → review-
Assignee | ||
Comment 5•12 years ago
|
||
STR: - apply the patch from comment 3 - call your voicemail - from a second phone call the b2g phone - tap the end and answer button Expected: - the voicemail call is ended - the incoming call is answered Actual: - the voicemail call is ended (I can hear the ringing tone on the second phone, confirming that it is indeed not picked up.) Tested on a unagi with a mozilla-beta gecko form yesterday. I also tried to wait for the ondisconnecting on the voicemail call before calling answer on the incoming call but no luck. I there a RIL-level issue preventing this patch from working?
Flags: needinfo?(htsai)
Comment 6•12 years ago
|
||
(In reply to Etienne Segonzac (:etienne) from comment #5) > STR: > - apply the patch from comment 3 > - call your voicemail > - from a second phone call the b2g phone > - tap the end and answer button > > Expected: > - the voicemail call is ended > - the incoming call is answered > > Actual: > - the voicemail call is ended > (I can hear the ringing tone on the second phone, confirming that it is > indeed not picked up.) > > Tested on a unagi with a mozilla-beta gecko form yesterday. > I also tried to wait for the ondisconnecting on the voicemail call before > calling answer on the incoming call but no luck. > > I there a RIL-level issue preventing this patch from working? Hi Etienne, The patch looks good, overall, but I realized that we should wait for the response from the network before sending the next request. So, if I modify the code as callToEnd.addEventListener('disconnected', function disconnected() { callToEnd.removeEventListener('disconnected', disconnected); callToAnswer.answer(); }); callToEnd.hangUp(); I can see what we are expecting.
Flags: needinfo?(htsai)
Assignee | ||
Comment 7•12 years ago
|
||
Thanks Hsin-Yi! Working perfectly.
Assignee: jones.chris.g → etienne
Attachment #691685 -
Attachment is obsolete: true
Attachment #692248 -
Flags: review?(francisco.jordano)
Comment 8•12 years ago
|
||
Comment on attachment 692248 [details]
Pointer to gaia PR 7019
Small change, love it :)
Attachment #692248 -
Flags: review?(francisco.jordano) → review+
Assignee | ||
Comment 9•12 years ago
|
||
https://github.com/mozilla-b2g/gaia/commit/8a3c61188a37d1a0961ccbf286d3d1c4c763a62a
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•