AVRCP: Update play position notification only to active device
If player update takes time in updating playback state and at the
same time Play interval timeout happens then we keep updating song
playposition for inactive device endlessly. From now onward we will
update song play position for only active device.
**When local streaming is happening and remote gets connected, also
player updates playback state as PLAYING(no other) then we did not
update mLastStateUpdate with system clock because of which remote
gets full song duration in GetPlayStatus response. Now initializing
this variable with system clock during connection itself.
CRs-Fixed: 2542784
Change-Id: I2962bcf4726b72df705d3990e37e5d9e96e7d09b
diff --git a/packages_apps_bluetooth_ext/src/avrcp/Avrcp_ext.java b/packages_apps_bluetooth_ext/src/avrcp/Avrcp_ext.java
index f27e387..f5cb2d3 100644
--- a/packages_apps_bluetooth_ext/src/avrcp/Avrcp_ext.java
+++ b/packages_apps_bluetooth_ext/src/avrcp/Avrcp_ext.java
@@ -2914,7 +2914,8 @@
mHandler.removeMessages(currMsgPlayIntervalTimeout);
if ((deviceFeatures[i].mCurrentDevice != null) &&
(deviceFeatures[i].mPlayPosChangedNT == AvrcpConstants_ext.NOTIFICATION_TYPE_INTERIM) &&
- (isPlayingState(deviceFeatures[i].mCurrentPlayState)) && !isPlayerPaused()) {
+ (isPlayingState(deviceFeatures[i].mCurrentPlayState)) &&
+ !isPlayerPaused() && deviceFeatures[i].isActiveDevice) {
Message msg = mHandler.obtainMessage(currMsgPlayIntervalTimeout, 0, 0,
deviceFeatures[i].mCurrentDevice);
long delay = deviceFeatures[i].mPlaybackIntervalMs;
@@ -3541,8 +3542,9 @@
//When A2dp playing on DUT and Remote got connected, send proper playstatus
if (isPlayingState(mCurrentPlayerState) &&
mA2dpService.isA2dpPlaying(device)) {
+ deviceFeatures[index].mLastStateUpdate = mLastStateUpdate;
deviceFeatures[index].mCurrentPlayState = mCurrentPlayerState;
- Log.i(TAG,"Send correct playstatus to remote when it gets connected: " +
+ Log.i(TAG,"Send correct playstatus and song position to remote when it gets connected: " +
deviceFeatures[index].mCurrentPlayState);
}
}