Closed
Bug 924787
Opened 11 years ago
Closed 11 years ago
[AudioManager] BT_SCO's volume can't be adjusted during voice call because of wrong check of mozsetting name.
Categories
(Firefox OS Graveyard :: General, defect)
Tracking
(blocking-b2g:koi+, firefox25 wontfix, firefox26 fixed, firefox27 fixed, b2g-v1.2 fixed)
RESOLVED
FIXED
blocking-b2g | koi+ |
People
(Reporter: mchen, Assigned: mchen)
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
1.01 KB,
patch
|
mchen
:
review+
|
Details | Diff | Splinter Review |
1. Enable BT and connected to BT headset (HFP). 2. Make a phone call to another side. 3. After picking up by remote side, try to change the volume by hardware key. Expected: the voice volume is changed. Actual: the voice volume is not changed.
Assignee | ||
Comment 1•11 years ago
|
||
The checking condition should be "accept to change volume only when settings name is bt_sco". The wrong condition now is "accept to change volume when settings name is not bt_sco."
Attachment #814795 -
Flags: review?(mwu)
Updated•11 years ago
|
Attachment #814795 -
Flags: review?(mwu) → review+
Assignee | ||
Comment 2•11 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=58fb5e2bfb00
Attachment #814795 -
Attachment is obsolete: true
Attachment #816505 -
Flags: review+
Assignee | ||
Comment 3•11 years ago
|
||
We need this patch for V1.2 or the BT SCO volume can't be adjusted during voice call state.
blocking-b2g: --- → koi?
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 4•11 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/f6f83e7e2bfa
Keywords: checkin-needed
Comment 5•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f6f83e7e2bfa
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Comment 6•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/778f774dbde2
status-b2g-v1.2:
--- → fixed
status-firefox25:
--- → wontfix
status-firefox26:
--- → fixed
status-firefox27:
--- → fixed
Whiteboard: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•