Closed Bug 1147123 Opened 9 years ago Closed 9 years ago

[CDMA] Cannot merge 3-way call

Categories

(Firefox OS Graveyard :: Gaia::Dialer, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:2.2+, b2g-v2.2 fixed, b2g-master fixed)

RESOLVED FIXED
2.2 S9 (3apr)
blocking-b2g 2.2+
Tracking Status
b2g-v2.2 --- fixed
b2g-master --- fixed

People

(Reporter: mschwart, Assigned: gsvelto)

References

Details

(Whiteboard: [caf priority: p2][CR 810202])

Attachments

(1 file, 1 obsolete file)

STR:
# Initiate a MO CDMA call
# Add a new call

Expected: Merge button should be available
Observed: No merge or swap button is available

This is a regression from https://bugzilla.mozilla.org/show_bug.cgi?id=977588#c112
blocking-b2g: --- → 2.2?
Whiteboard: [CR 810202]
Whiteboard: [CR 810202] → [caf priority: p2][CR 810202]
Comms triage: Regression from 2.1 on a use case which will impact CDMA phones
blocking-b2g: 2.2? → 2.2+
Taking this since I'm familiar with the topic.
Assignee: nobody → gsvelto
Target Milestone: --- → 2.2 S9 (3apr)
According to https://bugzilla.mozilla.org/show_bug.cgi?id=977588#c118 the mergeable/switchable flags from TelephonyCall should be used instead and these checks removed.
Working on this.
Status: NEW → ASSIGNED
Comment on attachment 8585765 [details] [review]
[gaia] gabrielesvelto:bug-1147123-cdma-3way-merge > mozilla-b2g:master

I can't test this patch since I don't have a CDMA device available and the emulator doesn't support it yet; but if the telephony code is doing its job properly on real devices this should work.

I've removed tests that didn't make sense anymore and updated the remaining ones.
Attachment #8585765 - Flags: review?(drs)
M4, could you please test the patch provided by Gabriele.
Flags: needinfo?(mschwart)
Comment on attachment 8585765 [details] [review]
[gaia] gabrielesvelto:bug-1147123-cdma-3way-merge > mozilla-b2g:master

See my comments on the PR. Would also like to get the results of partner testing before we land this.
Attachment #8585765 - Flags: review?(drs) → review-
Comment on attachment 8585765 [details] [review]
[gaia] gabrielesvelto:bug-1147123-cdma-3way-merge > mozilla-b2g:master

Addressed the review comments. I've moved around some tests and added more to better capture the changes I made. I hope this is good enough; I have to ask you to land this for me once it's r+'d and handle the upflit too.
Attachment #8585765 - Flags: review- → review?(drs)
Attachment #8585765 - Flags: review?(drs) → review+
Needinfo to request uplift. We should wait for M4's test results first, though.
Flags: needinfo?(drs)
Attachment #8586407 - Attachment is obsolete: true
I can merge calls with this patch.  I can't add another call once I'm in the conference state (someone has dropped off and I want to call them back) but that's another issue...
Flags: needinfo?(mschwart)
Doug, please uplift this patch to 2.2.
Comment on attachment 8585765 [details] [review]
[gaia] gabrielesvelto:bug-1147123-cdma-3way-merge > mozilla-b2g:master

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): Incomplete functionality from bug 977588.
[User impact] if declined: CDMA users will be unable to merge conference calls into 3-way calls.
[Testing completed]: m4 tried this. See comment 13.
[Risk to taking this patch] (and alternatives if risky): Medium as we can't test this ourselves since it's CDMA-only, and it's a fairly big patch, but there's no real alternative.
[String changes made]: None.
Flags: needinfo?(drs)
Attachment #8585765 - Flags: approval-gaia-v2.2?(bbajaj)
Bhavana, please get this uplifted to 2.2.
Flags: needinfo?(bbajaj)
Flags: needinfo?(bbajaj)
Attachment #8585765 - Flags: approval-gaia-v2.2?(bbajaj) → approval-gaia-v2.2+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: