commit b79ef23ed96ca130850ee00050ef803c49b367e9
author Eric Laurent <elaurent@google.com> Mon Mar 22 14:37:19 2021 +0100
committer Eric Laurent <elaurent@google.com> Mon Mar 29 19:31:05 2021 +0200
tree 4f31f7ff17b230ee53a1c8f12034872f948d3dda
parent b6521e0a8b7925e87a51f35241b9051be8d9fd00

AudioDeviceBroker: fix speakerphone state reporting

Send ACTION_SPEAKERPHONE_STATE_CHANGED only when the speakerphone
is actually turned ON/OFF instead of when requested ON/OFF.

Make sure we update the active communication device after the message
requesting the preferred phone strategy device is processed: when no
device is preferred, the actual active device is read back from audio
policy manager.

Also:
- Factored in and use consistent naming for methods dealing with communication
device selection
- Use cached SCO ON state for volume related decisions to avoid lock
contention

Bug: 171177848
Test: AudioManagerTest.testRouting AudioManagerTest.testSpeakerphoneIntent
Test: AudioCommunicationDeviceTest
Change-Id: Ic5e84a1f85f0fa5dac2abed34fc85a58f41e4816