commit 585c3a260b541207653931bb5259b60ab1aa10b4
author chandan mishra <chanmish@codeaurora.org> Tue May 21 16:40:28 2019 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Fri May 31 06:12:15 2019 -0700
tree 311adc1fb8fb6b076e581f0bf5c934c887a7f942
parent 4aa66d37589dd832ebd559f8d5d7f1669666638c

AVRCP: Updating mRemoteVolue before sending setAbsoluteVolume

To avoid setAbsoluteVolume call twice updating
mRemoteVolume with mLocalVolume before setVolumeNative
function call.

CRs-Fixed: 2454985
Change-Id: I2bff4f13b96687ec6487664af6f2d317116db4d9

packages_apps_bluetooth_ext/src/avrcp/Avrcp_ext.java

diff --git a/packages_apps_bluetooth_ext/src/avrcp/Avrcp_ext.java b/packages_apps_bluetooth_ext/src/avrcp/Avrcp_ext.java
index cd83150..6147857 100644
--- a/packages_apps_bluetooth_ext/src/avrcp/Avrcp_ext.java
+++ b/packages_apps_bluetooth_ext/src/avrcp/Avrcp_ext.java
@@ -816,9 +816,10 @@
                         remVol = convertToAudioStreamVolume(remVol);
                     }
                     Log.d(TAG,"vol = " + vol + "remVol = " + remVol);
-                    if(remVol != -1 && vol != remVol &&
+                    if (vol != remVol &&
                        deviceFeatures[deviceIndex].mCurrentDevice != null) {
                        setVolumeNative(volume , getByteAddress(deviceFeatures[deviceIndex].mCurrentDevice));
+                       deviceFeatures[deviceIndex].mRemoteVolume = volume;
                        if (deviceFeatures[deviceIndex].mCurrentDevice.isTwsPlusDevice()) {
                            AdapterService adapterService = AdapterService.getAdapterService();
                            BluetoothDevice peer_device =
@@ -1187,6 +1188,7 @@
                         complete and making synchronization to send only one setAbsolute volume
                         during connection*/
                         if(getVolume(deviceFeatures[deviceIndex].mCurrentDevice) != -1) {
+                            deviceFeatures[deviceIndex].mRemoteVolume = absVol;
                             setAbsVolumeFlag(deviceFeatures[deviceIndex].mCurrentDevice);
                             break;
                         }