Closed Bug 820642 Opened 12 years ago Closed 11 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)

ARM
Gonk (Firefox OS)
defect

Tracking

(blocking-basecamp:+)

VERIFIED FIXED
B2G C3 (12dec-1jan)
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.
blocking-basecamp: --- → ?
Hsin-Yi, tentatively assigning to you.  Please reassign as appropriate :).
Assignee: nobody → htsai
blocking-basecamp: ? → +
Priority: -- → P1
Status: UNCONFIRMED → NEW
Ever confirmed: true
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
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)
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-
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)
(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)
Thanks Hsin-Yi! Working perfectly.
Assignee: jones.chris.g → etienne
Attachment #691685 - Attachment is obsolete: true
Attachment #692248 - Flags: review?(francisco.jordano)
Comment on attachment 692248 [details]
Pointer to gaia PR 7019

Small change, love it :)
Attachment #692248 - Flags: review?(francisco.jordano) → review+
https://github.com/mozilla-b2g/gaia/commit/8a3c61188a37d1a0961ccbf286d3d1c4c763a62a
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Verified fixed Unagi build 20130103070201
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: