Closed Bug 816913 Opened 8 years ago Closed 8 years ago
B2G RIL: update phone
State when there's an incoming call
We doesn't change gAudioManager.phoneState to PHONE_STATE_RING_TONE correctly when there comes an incoming call. We should fix that.
Comment on attachment 687017 [details] [diff] [review] patch: update gAudioManager.phoneState as PHONE_STATE_RING_TONE Should address comments per discussion.
This patch gathered scatters of 'active call determination' in ril_worker and RadioInterfaceLayer. The determination was centralized in 'updateCallAudioState()'
Attachment #687017 - Attachment is obsolete: true
This is the correct v2.
Attachment #687654 - Attachment is obsolete: true
Attachment #687656 - Flags: review?(allstars.chh)
8 years ago
Attachment #687656 - Flags: review?(allstars.chh) → review+
This patch LGTM. AudioManager use MODE_RING to work-around the sound-leakage problem(for ex, answer MT call, the ringtone may delay several m-sec to stop).
(In reply to Randy Lin [:rlin] from comment #5) > This patch LGTM. AudioManager use MODE_RING to work-around the sound-leakage > problem(for ex, answer MT call, the ringtone may delay several m-sec to > stop). Randy, thanks for your feedback :)
bb? because we should fix the wrong phoneState.
Hsin-Yi, what will happen if we don't fix this?
Hi Andrew, If we don't fix this bug, there could be ringtone leakage problem. I.e. When you answer the incoming call by press the answer key, the ringetone would still play, and the remote party may hear that sound. Details as Randy explained in comment 5. Thanks!
blocking-basecamp: ? → -
Priority: -- → P4
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
Comment on attachment 687656 [details] [diff] [review] Patch v2 [Approval Request Comment] Bug caused by (feature/regressing bug #): User impact if declined: please see comment 9 Testing completed (on m-c, etc.): pushed to m-c already Risk to taking this patch (and alternatives if risky): low String or UUID changes made by this patch: no
Attachment #687656 - Flags: approval-mozilla-aurora?
Comment on attachment 687656 [details] [diff] [review] Patch v2 no bb+ but also doesn't look to affect mainstream FF so approving for uplift.
Attachment #687656 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.