Closed
Bug 1151992
Opened 9 years ago
Closed 9 years ago
Firefox Hello audio on Flame v2.2 is not routed to the loud speaker
Categories
(Firefox OS Graveyard :: Gaia::Loop, defect)
Tracking
(blocking-b2g:2.2+, firefox38 wontfix, firefox39 wontfix, firefox40 fixed, b2g-v2.2 verified, b2g-master fixed)
People
(Reporter: diego, Assigned: alwu)
References
Details
(Whiteboard: [caf priority: p2][CR 815031])
Attachments
(2 files, 1 obsolete file)
1.56 KB,
patch
|
alwu
:
review+
bajaj
:
approval-mozilla-b2g37+
|
Details | Diff | Splinter Review |
6.00 MB,
video/3gpp
|
Details |
The incoming sound during an audio/video session is instead routed to the phone's earpiece. You can hear sound if you put the phone up to your ear like a regular phone call. This is a regression from v2.1 were it sound came through the loud speaker.
Reporter | ||
Updated•9 years ago
|
blocking-b2g: --- → 2.2?
Reporter | ||
Updated•9 years ago
|
Whiteboard: [CR 815031]
Updated•9 years ago
|
Whiteboard: [CR 815031] → [caf priority: p2][CR 815031]
Reporter | ||
Updated•9 years ago
|
Summary: Firefox Hello audio on Flame v2.2 is not routed to the speaker → Firefox Hello audio on Flame v2.2 is not routed to the loud speaker
Comment 2•9 years ago
|
||
Hi Alastor, Please check this bug, thanks.
Flags: needinfo?(slee) → needinfo?(alwu)
Assignee | ||
Comment 5•9 years ago
|
||
[Root cause] Didn't check the call state, we used the "Media" usage to switch the speaker. [Solution] Since there was the voice-call audio track on the audio output, the routing would be set to receiver. The only way to change the routing to speaker is to use the "Communication" usage instead of "Media". --- If the try-server result is ok, I will ask for the review. https://treeherder.mozilla.org/#/jobs?repo=try&revision=ebdc386f0a0a
Reporter | ||
Comment 6•9 years ago
|
||
Comment on attachment 8590125 [details] [diff] [review] Use the communication usage to switch the routing Review of attachment 8590125 [details] [diff] [review]: ----------------------------------------------------------------- The patch works for me, thanks!
Attachment #8590125 -
Flags: feedback+
Reporter | ||
Updated•9 years ago
|
status-b2g-v2.2:
--- → affected
Assignee | ||
Comment 7•9 years ago
|
||
Comment on attachment 8590125 [details] [diff] [review] Use the communication usage to switch the routing Hi, Baku, Could you help me review this patch? Very appreciate :)
Attachment #8590125 -
Flags: review?(amarchesini)
Updated•9 years ago
|
Attachment #8590125 -
Flags: review?(amarchesini) → review+
Assignee | ||
Comment 8•9 years ago
|
||
Attachment #8590125 -
Attachment is obsolete: true
Attachment #8590692 -
Flags: review+
Assignee | ||
Comment 9•9 years ago
|
||
Comment on attachment 8590692 [details] [diff] [review] Use the communication usage to switch the routing. r=baku. [Approval Request Comment] Bug caused by (feature/regressing bug #): Regression bug User impact if declined: Can't hear sound from the speaker during the VOIP call Testing completed: Yes Risk to taking this patch (and alternatives if risky): None String or UUID changes made by this patch: None
Attachment #8590692 -
Flags: approval-mozilla-b2g37?
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 10•9 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/8fa8ef7b1d47
Keywords: checkin-needed
Comment 11•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8fa8ef7b1d47
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox40:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S10 (17apr)
Updated•9 years ago
|
Attachment #8590692 -
Flags: approval-mozilla-b2g37? → approval-mozilla-b2g37+
Comment 12•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/415551dbf781
Comment 13•9 years ago
|
||
This bug has been verified as "pass" on latest nightly build of Flame v2.2. STR: 1. Install Hello app and sign in a firefox account. 2. Make a audio/video call and other device receives this call. 3. Listen to the sound. Actual results: The sound is routed to the loud speaker. See attachment: verified_Flame_v2.2.3gp Reproduce rate: 0/10 Device: Flame v2.2 (Verified) Build ID 20150719002503 Gaia Revision e1e6317f17a840b19af9dbb25f5a771d8d9fa161 Gaia Date 2015-07-15 21:05:11 Gecko Revision https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/e2d1f1f55803 Gecko Version 37.0 Device Name flame Firmware(Release) 4.4.2 Firmware(Incremental) eng.cltbld.20150719.041302 Firmware Date Sun Jul 19 04:13:13 EDT 2015 Bootloader L1TC000118D0 Note: I can't verify this bug on latest Flame master, because it can't login the firefox account in Loop app (blocked by Bug 1168173).
Comment 14•9 years ago
|
||
Updated•9 years ago
|
QA Whiteboard: [MGSEI-Triage+]
Comment 15•9 years ago
|
||
According to "https://bugzilla.mozilla.org/show_bug.cgi?id=1168173#c8", it is 'WON'T FIX' since we're not supporting the Hello Firefox OS application anymore, so change status to "verified".
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•