[Loop] MO Ringtone always plays over speaker even wired headset was connected

VERIFIED FIXED

Status

Firefox OS
Gaia::Loop
--
major
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: Jay, Assigned: Carmen Jimenez Cabezas)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mobile app][blocking][patch available][caf priority: p1][CR 735661])

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Pre-Conditions:
1. Insert stereo wired headset 

Steps to reproduce:
1. Launch FirefoxHello app
2. Now start MO VT call from the device

Issue:
Observed MO calling tone is not routing to stereo wired headset

Expected behavior:
MO calling tone should route to stereo wired headset

Typically, the MT ringtone plays out on speaker but the MO calling tone plays over wired headset.
(Reporter)

Comment 1

3 years ago
Can Tef help to take a look at this?
Flags: needinfo?(oteo)

Updated

3 years ago
Whiteboard: [CR 735661]

Updated

3 years ago
Whiteboard: [CR 735661] → [caf priority: p1][CR 735661]
Sure! including it in the meta so QA team can confirm it
Blocks: 1036490
Flags: needinfo?(oteo)
Yep, this is pretty bad. When device is going to start a Video Call, any sound is by default sent to the speaker, there are two situations where this can happen:

1 - When the user starts a video call (the one reported in this bug), the ringback tones are sent to the speaker and the conversation audio too.
2 - When the user decides to accept a call as a video call. In this case, the conversation audio is routed to the speaker.

We should add an extra condition that is checking if headsets are available and in that case either in 1/ or 2/ we should send the sound to the headset (not to the speaker)
Whiteboard: [caf priority: p1][CR 735661] → [mobile app][blocking][caf priority: p1][CR 735661]

Updated

3 years ago
Severity: normal → major

Updated

3 years ago
Blocks: 1025317
(Assignee)

Updated

3 years ago
Assignee: nobody → carmen.jimenezcabezas

Comment 4

3 years ago
[Blocking Requested - why for this release]:
blocking-b2g: --- → 2.1?
Hi Inder, this is not a gecko/gaia bug so the blocking-b2g flag does not apply. It needs to be fixed in the Loop mobile aplicación and has already been triaged and prioritized as a blocker bug.
blocking-b2g: 2.1? → ---
Status: NEW → ASSIGNED
(Assignee)

Comment 6

3 years ago
Created attachment 8505010 [details] [review]
V1 Proposed patch
Whiteboard: [mobile app][blocking][caf priority: p1][CR 735661] → [mobile app][blocking][patch available][caf priority: p1][CR 735661]
(Assignee)

Updated

3 years ago
Attachment #8505010 - Flags: review?(josea.olivera)
Attachment #8505010 - Flags: review?(crdlc)
Comment on attachment 8505010 [details] [review]
V1 Proposed patch

Thanks Carmen, as we commented offline the patch needs some changes. Request review at us when you are done please.
Attachment #8505010 - Flags: review?(josea.olivera)
Attachment #8505010 - Flags: review?(crdlc)
(Assignee)

Updated

3 years ago
Attachment #8505010 - Flags: review?(josea.olivera)
Attachment #8505010 - Flags: review?(crdlc)
Comment on attachment 8505010 [details] [review]
V1 Proposed patch

LGTM, thanks
Attachment #8505010 - Flags: review?(crdlc) → review+
Attachment #8505010 - Flags: review?(josea.olivera) → review+
(Assignee)

Comment 9

3 years ago
https://github.com/mozilla-b2g/firefoxos-loop-client/commit/d56fe348f37ed3a622a8b78ebe0063903623bd39
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED

Comment 10

3 years ago
Verified on loop version 1.1: 38eadf0
Flame:flame-KK.user.v2.0.188based.B-66.Gecko-567c7eb.Gaia-ab83632
FireE:firee-kk-v2.0-SW2E5-4 

Tone is routing to stereo wired headset, and when incoming call is receiving, tone is heard on both, speaker and headset, as in GSM calls.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.