Closed Bug 825635 Opened 10 years ago Closed 10 years ago
Volume control stops working in FM when headset is unplugged and plugged
The volume control stops working after following these steps: 1) Boot up the phone 2) Start the FM radio 3) Plug in a headset 4) Listen to the radio for a moment and adjust volume (note: volume actually changes) 5) Unplug the headset 6) Plug the headset back in 7) Adjust the volume and note that it does not result in a volume change any longer.
Assignee: nobody → alive
blocking-basecamp: ? → +
Priority: -- → P3
Target Milestone: --- → B2G C4 (2jan on)
I think this is more like a platform bug. Redirect to Randy. It seems when FM is changing its state, the audio-channel-changed is not fired at step(7).
Assignee: alive → rlin
Summary: Volume control stops working → Volume control stops working in FM when headset is unplugged and plugged
If FM radio turn off, the mCurrentHigherChannel would keep in AUDIO_CHANNEL_CONTENT,so if the next FM radio turning on, the getMuted function wouldn't broadcast audio channel change, cause by this statement:higher == mCurrentHigherChannel == AUDIO_CHANNEL_CONTENT. Music isn't has this problem. I found the getMuted would check normal channel first and that would replace mCurrentHigherChannel value. then check CONTENT channel again.
Attachment #697340 - Flags: review?(amarchesini)
Attachment #697340 - Flags: review?(amarchesini) → review+
check in patch
Attachment #697340 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
This patch needs to phase in b2g-18 trunk.
In Unagi build 20130125070201 Version 18.0 This bug is still occurring. If I turn the radio on then press the power button to make the screen dark, The Hardware volume buttons are not reliable. I was able to turn the volume down but could not turn the volume back up. I've already written a bug about it but it was duped and I know people hate reopening bugs. So I'll just comment and leave it up to the powers that be. I'm running Testrun 3 and it fails testcase https://moztrap.mozilla.org/runtests/run/702/env/296/?pagenumber=1&pagesize=20&sortfield=order&sortdirection=asc&filter-id=3933&filter-suite=168 Build ID: 20130125070201 Kernel: Dec 5
Additional info Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/94a2d6fcdfde Gaia: 6369dbf33b622faf4b4d176fed30b77c5c319dfc
You need to log in before you can comment on or make changes to this bug.